----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34816/ -----------------------------------------------------------
(Updated May 29, 2015, 6:18 p.m.) Review request for mesos, Jie Yu, Niklas Nielsen, Szymon Konefal, and Vinod Kone. Changes ------- s/TestResourceEstimator/MockResourceEstimator/ Without that change it will conflict with future TestResourceEstimator ModuleID enum. Repository: mesos Description (updated) ------- This patch prepares Resource Estimator tests for modularized ResourceEstimator. It includes: 1. Made oversubscription_test to be typed_test. 2. TestResourceEstimator changed to be a templated MockResourceEstimator with mocked methods - to inject resources and test plumbing between RE and Slave. Diffs (updated) ----- src/tests/mesos.hpp b8f7a2f9236166e42421d926718af8d45e857eba src/tests/oversubscription_tests.cpp 75c25b04c1e6a8e0e7e8fd55440743fe1699af88 Diff: https://reviews.apache.org/r/34816/diff/ Testing ------- make check Thanks, Bartek Plotka
