trackler 2.1.0.2 → 2.1.0.3

data/tracks/java/exercises/allergies/src/test/java/AllergiesTest.java

@@ -20,7 +20,7 @@ public class AllergiesTest {
         assertEquals(false, allergies.isAllergicTo(Allergen.CATS));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void allergicToEggs() {
         Allergies allergies = new Allergies(1);
@@ -28,7 +28,7 @@ public class AllergiesTest {
         assertEquals(true, allergies.isAllergicTo(Allergen.EGGS));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void allergicToPeanuts() {
         Allergies allergies = new Allergies(2);
@@ -36,7 +36,7 @@ public class AllergiesTest {
         assertEquals(true, allergies.isAllergicTo(Allergen.PEANUTS));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void allergicToShellfish() {
         Allergies allergies = new Allergies(4);
@@ -44,7 +44,7 @@ public class AllergiesTest {
         assertEquals(true, allergies.isAllergicTo(Allergen.SHELLFISH));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void allergicToStrawberries() {
         Allergies allergies = new Allergies(8);
@@ -52,7 +52,7 @@ public class AllergiesTest {
         assertEquals(true, allergies.isAllergicTo(Allergen.STRAWBERRIES));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void allergicToTomatoes() {
         Allergies allergies = new Allergies(16);
@@ -60,7 +60,7 @@ public class AllergiesTest {
         assertEquals(true, allergies.isAllergicTo(Allergen.TOMATOES));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void allergicToChocolate() {
         Allergies allergies = new Allergies(32);
@@ -68,7 +68,7 @@ public class AllergiesTest {
         assertEquals(true, allergies.isAllergicTo(Allergen.CHOCOLATE));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void allergicToPollen() {
         Allergies allergies = new Allergies(64);
@@ -76,7 +76,7 @@ public class AllergiesTest {
         assertEquals(true, allergies.isAllergicTo(Allergen.POLLEN));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void allergicToCats() {
         Allergies allergies = new Allergies(128);
@@ -84,7 +84,7 @@ public class AllergiesTest {
         assertEquals(true, allergies.isAllergicTo(Allergen.CATS));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void isAllergicToEggsInAdditionToOtherStuff() {
         Allergies allergies = new Allergies(5);
@@ -94,7 +94,7 @@ public class AllergiesTest {
         assertEquals(false, allergies.isAllergicTo(Allergen.STRAWBERRIES));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void noAllergies() {
         Allergies allergies = new Allergies(0);
@@ -102,7 +102,7 @@ public class AllergiesTest {
         assertEquals(0, allergies.getList().size());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void isAllergicToJustEggs() {
         Allergies allergies = new Allergies(1);
@@ -111,7 +111,7 @@ public class AllergiesTest {
         assertEquals(expectedAllergens, allergies.getList());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void isAllergicToJustPeanuts() {
         Allergies allergies = new Allergies(2);
@@ -120,7 +120,7 @@ public class AllergiesTest {
         assertEquals(expectedAllergens, allergies.getList());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void isAllergicToJustStrawberries() {
         Allergies allergies = new Allergies(8);
@@ -129,7 +129,7 @@ public class AllergiesTest {
         assertEquals(expectedAllergens, allergies.getList());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void isAllergicToEggsAndPeanuts() {
         Allergies allergies = new Allergies(3);
@@ -141,7 +141,7 @@ public class AllergiesTest {
         assertEquals(expectedAllergens, allergies.getList());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void isAllergicToEggsAndShellfish() {
         Allergies allergies = new Allergies(5);
@@ -153,7 +153,7 @@ public class AllergiesTest {
         assertEquals(expectedAllergens, allergies.getList());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void isAllergicToLotsOfStuff() {
         Allergies allergies = new Allergies(248);
@@ -168,7 +168,7 @@ public class AllergiesTest {
         assertEquals(expectedAllergens, allergies.getList());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void isAllergicToEverything() {
         Allergies allergies = new Allergies(255);
@@ -186,7 +186,7 @@ public class AllergiesTest {
         assertEquals(expectedAllergens, allergies.getList());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void ignoreNonAllergenScoreParts() {
         Allergies allergies = new Allergies(509);

data/tracks/java/exercises/anagram/src/test/java/AnagramTest.java

@@ -16,7 +16,7 @@ public class AnagramTest {
         assertTrue(detector.match(Arrays.asList("hello", "world", "zombies", "pants")).isEmpty());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testDetectSimpleAnagram() {
         Anagram detector = new Anagram("ant");
@@ -25,7 +25,7 @@ public class AnagramTest {
         assertThat(anagram.size(), is(1));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testDetectLongerAnagram() {
         Anagram detector = new Anagram("listen");
@@ -33,7 +33,7 @@ public class AnagramTest {
         assertThat(anagrams, hasItem("inlets"));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testDetectMultipleAnagrams() {
         Anagram detector = new Anagram("master");
@@ -41,7 +41,7 @@ public class AnagramTest {
         assertThat(anagrams, allOf(hasItem("maters"), hasItem("stream")));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testDetectMultipleAnagramsForLongerWord() {
         Anagram detector = new Anagram("allergy");
@@ -49,14 +49,14 @@ public class AnagramTest {
         assertThat(anagrams, allOf(hasItem("gallery"), hasItem("largely"), hasItem("regally")));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testDoesNotConfuseDifferentDuplicates() {
         Anagram detector = new Anagram("galea");
         assertTrue(detector.match(Arrays.asList("eagle")).isEmpty());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testIdenticalWordIsNotAnagram() {
         Anagram detector = new Anagram("corn");
@@ -65,35 +65,35 @@ public class AnagramTest {
         assertThat(anagrams.size(), is(1));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testIdenticalWordRepeatedIsNotAnagram() {
         Anagram detector = new Anagram("go");
         assertTrue(detector.match(Arrays.asList("go Go GO")).isEmpty());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testCapitalWordIsNotOwnAnagram() {
         Anagram detector = new Anagram("BANANA");
         assertTrue(detector.match(Arrays.asList("Banana")).isEmpty());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testEliminateAnagramsWithSameChecksum() {
         Anagram detector = new Anagram("mass");
         assertTrue(detector.match(Arrays.asList("last")).isEmpty());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testEliminateAnagramSubsets() {
         Anagram detector = new Anagram("good");
         assertTrue(detector.match(Arrays.asList("dog", "goody")).isEmpty());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testCaseInsensitiveWhenSubjectStartsWithUpperCaseLetter() {
         Anagram detector = new Anagram("Orchestra");
@@ -101,7 +101,7 @@ public class AnagramTest {
         assertThat(anagrams, hasItem("carthorse"));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testCaseInsensitiveWhenAnagramStartsWithUpperCaseLetter() {
         Anagram detector = new Anagram("orchestra");
@@ -109,7 +109,7 @@ public class AnagramTest {
         assertThat(anagrams, hasItem("Carthorse"));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testCaseInsensitiveWhenBothAnagramAndSubjectStartWithUpperCaseLetter() {
         Anagram detector = new Anagram("Orchestra");
@@ -117,14 +117,14 @@ public class AnagramTest {
         assertThat(anagrams, hasItem("Carthorse"));
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testWordIsNotItsOwnAnagram() {
         Anagram detector = new Anagram("banana");
         assertTrue(detector.match(Arrays.asList("Banana")).isEmpty());
     }
 
-    @Ignore
+    @Ignore("Remove to run test")
     @Test
     public void testAnagramMustUseAllLettersExactlyOnce() {
         Anagram detector = new Anagram("tapper");

data/tracks/lisp/docs/INSTALLATION.md

@@ -8,29 +8,23 @@ brew install sbcl
 brew install clisp
 ```
 
-(See their homepages for instructions for installing on other
-platforms.) 
+See their homepages for instructions for installing on other
+platforms.
 
-Then install [QuickLisp](http://www.quicklisp.org/beta/#installation)
-and download
-[http://beta.quicklisp.org/quicklisp.lisp](http://beta.quicklisp.org/quicklisp.lisp).
+We will use [QuickLisp](http://www.quicklisp.org/beta/#installation) for Lisp library management. To install QuickLisp: 
 
 ```bash
 curl -O http://beta.quicklisp.org/quicklisp.lisp
 ```
 
-Load it into your running Lisp implementation, and type:
+Now launch lisp, and copy-paste the following expressions to finish QuickLisp installation:
 
 ```lisp
-(quicklisp-quickstart:install)
+> (quicklisp-quickstart:install)  ;; this will install quicklisp
+> (ql:add-to-init-file)           ;; this will add quicklisp setup to your init file (recommended)
 ```
 
-To add the startup code to your init file so that the next time you
-start the lisp interpreter QuickLisp will be loaded, type:
-
-```lisp
-(ql:add-to-init-file)
-```
+See the documentation of your lisp implementation on how to load quicklisp.lisp into lisp.
 
 Optionally load lisp-unit:
 

data/tracks/python/exercises/bracket-push/bracket_push_test.py

@@ -3,7 +3,7 @@ import unittest
 from bracket_push import check_brackets
 
 
-# test cases adapted from `x-common//canonical-data.json` @ version: 1.0.0
+# test cases adapted from `x-common//canonical-data.json` @ version: 1.1.0
 
 class BracketPushTests(unittest.TestCase):
     def test_paired_square_brackets(self):
@@ -18,6 +18,9 @@ class BracketPushTests(unittest.TestCase):
     def test_wrong_ordered_brackets(self):
         self.assertEqual(check_brackets("}{"), False)
 
+    def test_wrong_closing_bracket(self):
+        self.assertEqual(check_brackets("{]"), False)
+
     def test_paired_with_whitespace(self):
         self.assertEqual(check_brackets("{ }"), True)
 

data/tracks/python/exercises/isogram/isogram_test.py

@@ -1,8 +1,9 @@
 import unittest
+
 from isogram import is_isogram
 
 
-# test cases adapted from `x-common//canonical-data.json` @ version: 1.0.0
+# test cases adapted from `x-common//canonical-data.json` @ version: 1.1.0
 
 class TestIsogram(unittest.TestCase):
 
@@ -30,6 +31,9 @@ class TestIsogram(unittest.TestCase):
     def test_made_up_name_that_is_an_isogram(self):
         self.assertTrue(is_isogram("Emily Jung Schwartzkopf"))
 
+    def test_duplicated_character_in_the_middle(self):
+        self.assertFalse(is_isogram("accentor"))
+
 
 if __name__ == '__main__':
     unittest.main()

data/tracks/ruby/README.md

@@ -64,7 +64,7 @@ Note that flags which have an attached value, like above, must take the form
 
 ### Generated Test Suites
 
-If you find an `example.tt` file in a problem directory, then the test suite is
+If you find a `<problem_name>_cases.rb` file in `lib/`, then the test suite is
 generated from shared data, which can be found in the exercise definition in the [x-common][]
 repository.
 
@@ -94,7 +94,8 @@ bin/generate $PROBLEM
 
 #### Changing a Generated Exercise
 
-The `$PROBLEM/$PROBLEM_test.rb` will never be edited directly.
+Do not edit `$PROBLEM/$PROBLEM_test.rb`. Any changes you make will be overwritten when
+the file is generated the next time.
 
 There are two reasons why a test suite might change:
 
@@ -114,23 +115,14 @@ the exercise, which lives in the x-common repository.
 This change will need to be submitted as a pull request to the x-common repository. This pull
 request needs to be merged before you can regenerate the exercise.
 
-Changes that don't have to do directly with the test inputs and outputs, will
-most likely be made to `lib/$PROBLEM_cases.rb` but may also be made to
-`exercises/$PROBLEM/example.tt`. Then you can regenerate the exercise with
+Changes that don't have to do directly with the test inputs and outputs should
+be made to `lib/$PROBLEM_cases.rb`. Then you can regenerate the exercise with
 `bin/generate $PROBLEM`.
 
 #### Implementing a Generator
 
-You will need to implement two files and a directory to create a generator:
-
-1. `lib/$PROBLEM_cases.rb` - the logic for turning the data into tests.
-1. `exercises/$PROBLEM/example.tt` - the Erb template for the test file, `$PROBLEM_test.rb`.
-1. `exercises/$PROBLEM/.meta/` - metadata directory, currently contains version file
-
-You will not need to touch the top-level script, `bin/generate`.
-
-The `bin/generate` command relies on some common logic implemented in `lib/generator.rb`.
-You probably won't need to touch that, either.
+You will need to write code to produce the code that goes inside the test methods. Your
+code will live in `lib/$PROBLEM_cases.rb`.
 
 `lib/$PROBLEM_cases.rb` contains a derived class of `ExerciseCase` (in `lib/generator/exercise_cases.rb`)
 which wraps the JSON for a single test case. The default version looks something like this:
@@ -149,7 +141,7 @@ end
 Instead of `ProblemName` use the CamelCased name of the actual problem. This is important, since
 the generator script will infer the name of the class from the argument that is passed.
 
-This class must provide the methods used by `example.tt`. The base class provides methods
+This class must provide the methods used by `lib/generator/test_template.erb`. The base class provides methods
 for the default template for everything except `workload`.
 
 `workload` generates the code for the body of a test, including the assertion
@@ -157,34 +149,16 @@ and any setup required. The base class provides a variety of assertion and
 helper methods. Beyond that, you can implement any helper methods that you need
 as private methods in your derived class. See below for more information about [the intention of workload](#workload-philosophy)
 
-Finally, you need to create a text template, `example.tt`, as the bases for the test suite.
+If you really must add additional logic to the view template, you can use a custom
+template. Copy `lib/generator/test_template.erb` to your exercise directory, name it
+`exercise.tt`, and customize.
 
-Start with the following boilerplate, and adjust as necessary. Remember, however, to strive
-to keep logic out of views.
+You will not need to touch the top-level script, `bin/generate`.
 
-```
-#!/usr/bin/env ruby
-gem 'minitest', '>= 5.0.0'
-require 'minitest/autorun'
-require_relative '$PROBLEM'
-
-# Common test data version: <%= abbreviated_commit_hash %>
-class ProblemNameTest < Minitest::Test
-<% test_cases.each do |test_case| %>
-  def <%= test_case.name %>
-    <%= test_case.skipped %>
-    <%= test_case.workload %>
-  end
+The `bin/generate` command relies on some common logic implemented in `lib/generator.rb`.
+You won't need to touch that, either.
 
-<% end %>
-<%= IO.read(XRUBY_LIB + '/bookkeeping.md') %>
 
-  def test_bookkeeping
-    skip
-    assert_equal <%= version %>, BookKeeping::VERSION
-  end
-end
-```
 ### Workload philosophy.
 
 Prioritize educational value over expert comprehension and make sure that