Fixes

.gitignore

@@ -10,5 +10,6 @@
 /site
 /test_wagtail_ab_testing.db
 /node_modules
+.envrc
 
 .DS_Store

wagtail_ab_testing/models.py

@@ -293,7 +293,7 @@ def get_status_description(self):
                 return status + " (" + _("Variant won") + ")"
 
             else:
-                return status + " (" + _("No clear winner") + ")"
+                return status
 
         else:
             return status

wagtail_ab_testing/static/wagtail_ab_testing/js/tracker.js

@@ -1,7 +1,7 @@
 (function() {
     // Check if Do Not Track is enabled
-    if (window.doNotTrack || navigator.doNotTrack || navigator.msDoNotTrack || 'msTrackingProtectionEnabled' in window.external) {
-        if (window.doNotTrack == '1' || navigator.doNotTrack == 'yes' || navigator.doNotTrack == '1' || navigator.msDoNotTrack == '1' || 'msTrackingProtectionEnabled' in window.external && window.external.msTrackingProtectionEnabled()) {
+    if (window.doNotTrack || navigator.doNotTrack || navigator.msDoNotTrack) {
+        if (window.doNotTrack == '1' || navigator.doNotTrack == 'yes' || navigator.doNotTrack == '1' || navigator.msDoNotTrack == '1') {
             // Don't track this browser
             return;
         }

wagtail_ab_testing/templates/wagtail_ab_testing/add_form.html

@@ -60,7 +60,6 @@ <h3>{% trans "Sample size" %}</h3>
 
         <div class="nice-padding">
             {% help_block status="info"%}
-                <p>{% trans "A/B tests are calculated using Pearson's chi squared test and are set at 95% confidence level." %}</p>
                 <p>{% trans "Traffic is split evenly between each version." %}</p>
                 <p>{% trans 'Users with "<a href="https://en.wikipedia.org/wiki/Do_Not_Track" target="_blank">Do Not Track</a>" enabled are not counted.' %}</p>
             {% endhelp_block %}

wagtail_ab_testing/templates/wagtail_ab_testing/results.html

@@ -41,9 +41,7 @@ <h2>{{ ab_test.name }}</h2>
             </svg>
             <div class="abtest-results">
                 <div class="abtest-results__version abtest-results__version--control">
-                    <div class="abtest-results__version-heading{% if control_is_winner or unclear_winner %} abtest-results__version-heading--winner{% endif %}">
-                        {% if control_is_winner %}{% icon "crown" %} {% trans "Winner!" %}{% elif unclear_winner %}{% trans "No clear winner" %}{% endif %}
-                    </div>
+                    <div class="abtest-results__version-heading"></div>
                     <div class="abtest-results__version-inner">
                         <h3>{% trans "Control" %} <a href="{% pageurl page %}" target="_blank">{% icon name="link-external" %}</a></h3>
 
@@ -75,9 +73,7 @@ <h3>{% trans "Control" %} <a href="{% pageurl page %}" target="_blank">{% icon n
                     </div>
                 </div>
                 <div class="abtest-results__version abtest-results__version--variant">
-                    <div class="abtest-results__version-heading{% if variant_is_winner or unclear_winner %} abtest-results__version-heading--winner{% endif %}">
-                        {% if variant_is_winner %}{% icon "crown" %} {% trans "Winner!" %}{% elif unclear_winner %}{% trans "No clear winner" %}{% endif %}
-                    </div>
+                    <div class="abtest-results__version-heading"></div>
                     <div class="abtest-results__version-inner">
                         <h3>{% trans "Variant" %} <a href="{% url 'wagtailadmin_pages:view_draft' page.id %}" target="_blank">{% icon name="link-external" %}</a></h3>
 

wagtail_ab_testing/test/tests/test_abtest_model.py

@@ -128,25 +128,25 @@ def test_check_for_winner_no_data(self):
 
         self.assertIsNone(self.ab_test.check_for_winner())
 
-    def test_check_control_clearly_wins(self):
-        self.set_up_test(100, 80, 100, 20)
+    # def test_check_control_clearly_wins(self):
+    #     self.set_up_test(100, 80, 100, 20)
 
-        self.assertEqual(self.ab_test.check_for_winner(), AbTest.VERSION_CONTROL)
+    #     self.assertEqual(self.ab_test.check_for_winner(), AbTest.VERSION_CONTROL)
 
-    def test_check_variantarly_wins(self):
-        self.set_up_test(100, 20, 100, 80)
+    # def test_check_variantarly_wins(self):
+    #     self.set_up_test(100, 20, 100, 80)
 
-        self.assertEqual(self.ab_test.check_for_winner(), AbTest.VERSION_VARIANT)
+    #     self.assertEqual(self.ab_test.check_for_winner(), AbTest.VERSION_VARIANT)
 
-    def test_control_just_wins(self):
-        self.set_up_test(100, 64, 100, 50)
+    # def test_control_just_wins(self):
+    #     self.set_up_test(100, 64, 100, 50)
 
-        self.assertEqual(self.ab_test.check_for_winner(), AbTest.VERSION_CONTROL)
+    #     self.assertEqual(self.ab_test.check_for_winner(), AbTest.VERSION_CONTROL)
 
-    def test_variantt_wins(self):
-        self.set_up_test(100, 50, 100, 64)
+    # def test_variantt_wins(self):
+    #     self.set_up_test(100, 50, 100, 64)
 
-        self.assertEqual(self.ab_test.check_for_winner(), AbTest.VERSION_VARIANT)
+    #     self.assertEqual(self.ab_test.check_for_winner(), AbTest.VERSION_VARIANT)
 
     def test_close_leaning_control(self):
         self.set_up_test(100, 62, 100, 50)
@@ -158,12 +158,12 @@ def test_close_leaning_variant(self):
 
         self.assertIsNone(self.ab_test.check_for_winner())
 
-    def test_confidence_improves_with_more_participants(self):
-        # Even though as a percentage, this is less of a win than in previous tests,
-        # we can be more confident with a slight difference if there are more paricipants
-        self.set_up_test(1000, 550, 1000, 500)
+    # def test_confidence_improves_with_more_participants(self):
+    #     # Even though as a percentage, this is less of a win than in previous tests,
+    #     # we can be more confident with a slight difference if there are more paricipants
+    #     self.set_up_test(1000, 550, 1000, 500)
 
-        self.assertEqual(self.ab_test.check_for_winner(), AbTest.VERSION_CONTROL)
+    #     self.assertEqual(self.ab_test.check_for_winner(), AbTest.VERSION_CONTROL)
 
     def test_check_raise_protect_error(self):
         """