From 0e5f82df4775c8035849b874e993b98fb66a4f8b Mon Sep 17 00:00:00 2001
From: Cam Saul <cammsaul@gmail.com>
Date: Tue, 2 Apr 2019 18:42:02 -0700
Subject: [PATCH] Add tests to check DisallowConcurrentExecution annotations
are present
---
test/metabase/task/sync_databases_test.clj | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/test/metabase/task/sync_databases_test.clj b/test/metabase/task/sync_databases_test.clj
index 5ba76457ea4..b5ae6a245ff 100644
--- a/test/metabase/task/sync_databases_test.clj
+++ b/test/metabase/task/sync_databases_test.clj
@@ -3,7 +3,7 @@
testing is part of `metabase.models.database`, so there's an argument to be made that these sorts of tests could
just as easily belong to a `database-test` namespace."
(:require [clojure.string :as str]
- [expectations :refer :all]
+ [expectations :refer [expect]]
[metabase.models.database :refer [Database]]
[metabase.task.sync-databases :as sync-db]
[metabase.test.util :as tu]
@@ -14,6 +14,13 @@
[toucan.util.test :as tt])
(:import [metabase.task.sync_databases SyncAndAnalyzeDatabase UpdateFieldValues]))
+;; make sure our annotations are present
+(expect
+ (.isAnnotationPresent SyncAndAnalyzeDatabase org.quartz.DisallowConcurrentExecution))
+
+(expect
+ (.isAnnotationPresent UpdateFieldValues org.quartz.DisallowConcurrentExecution))
+
(defn- replace-trailing-id-with-<id> [s]
(str/replace s #"\d+$" "<id>"))
--
GitLab