From ae4e71f550c16b02fd5b53969c7d5b8153b06eb0 Mon Sep 17 00:00:00 2001 From: Alexis Hernandez Date: Fri, 19 Jan 2018 23:26:39 -0600 Subject: [PATCH] alerts-server: Fix non-deterministic test on FixedPriceAlertPostgresDataHandlerSpec --- .../FixedPriceAlertPostgresDataHandlerSpec.scala | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/alerts-server/test/com/alexitc/coinalerts/data/anorm/FixedPriceAlertPostgresDataHandlerSpec.scala b/alerts-server/test/com/alexitc/coinalerts/data/anorm/FixedPriceAlertPostgresDataHandlerSpec.scala index 2cb6deb..5690683 100644 --- a/alerts-server/test/com/alexitc/coinalerts/data/anorm/FixedPriceAlertPostgresDataHandlerSpec.scala +++ b/alerts-server/test/com/alexitc/coinalerts/data/anorm/FixedPriceAlertPostgresDataHandlerSpec.scala @@ -103,13 +103,17 @@ class FixedPriceAlertPostgresDataHandlerSpec extends PostgresDataHandlerSpec { result.exists(_.id == alert.id) mustEqual true } - // TODO: Fix non-deterministic test "retrieve several pending alerts" in { - val createAlertModel = CreateFixedPriceAlertModel(RandomDataGenerator.item(currencies).id, true, BigDecimal("5000.00"), None) + val selectedCurrencies = RandomDataGenerator.uniqueItems(currencies, 3) + val currency1 = selectedCurrencies.head + val currency2 = selectedCurrencies(1) + val currency3 = selectedCurrencies(2) + + val createAlertModel = CreateFixedPriceAlertModel(currency1.id, true, BigDecimal("5000.00"), None) val createAlert1 = createAlertModel val createAlert2 = createAlertModel.copy(isGreaterThan = false) - val createAlert3 = createAlertModel.copy(exchangeCurrencyId = RandomDataGenerator.item(currencies).id) - val createAlert4 = createAlertModel.copy(exchangeCurrencyId = RandomDataGenerator.item(currencies).id) + val createAlert3 = createAlertModel.copy(exchangeCurrencyId = currency2.id) + val createAlert4 = createAlertModel.copy(exchangeCurrencyId = currency3.id) val user = createUnverifiedUser() val alert1 = alertPostgresDataHandler.create(createAlert1, user.id).get