fixup! fixup! fixup! Introduce new class: Pipenv

Author: nre-ableton

src/com/ableton/Pipenv.groovy

@@ -21,30 +21,23 @@ class Pipenv implements Serializable {
    * other words, it runs {@code pipenv install --dev}. Also, it removes the virtualenv
    * after the last Python version has been run.
    *
-   * @param args Map of arguments. Valid arguments include:
-   *             <ul>
-   *               <li>
-   *                 {@code pythonVersions}: List of Python versions to run the command
-   *                 with. This argument is passed to Pipenv via {@code pipenv --python}.
-   *                 See {@code pipenv --help} for supported syntax.
-   *                 <strong>Required value, may not be null!</strong>
-   *               </li>
-   *             </ul>
+   * @param pythonVersions List of Python versions to run the command with. This argument
+   *                       is passed to Pipenv via {@code pipenv --python}. See
+   *                       {@code pipenv --help} for supported syntax.
    * @param body Closure body to execute. The closure body is passed the python version
    *             as an argument.
    * @return Map of return values. The keys in the map correspond to the Python versions
    *         given in {@code args.pythonVersions}, and the values are the result of
    *         executing the closure body.
    */
-  Map runWith(Map args, Closure body) {
+  Map runWith(List pythonVersions, Closure body) {
     assert script
-    assert args
-    assert args.pythonVersions
+    assert pythonVersions
 
     Map result = [:]
 
     try {
-      args.pythonVersions.each { python ->
+      pythonVersions.each { python ->
         script.sh "pipenv install --dev --python ${python}"
         result[python] = body(python)
       }

test/com/ableton/PipenvTest.groovy

@@ -40,7 +40,7 @@ class PipenvTest extends BasePipelineTest {
     }
 
     int numCalls = 0
-    Map result = new Pipenv(script: script).runWith(pythonVersions: pythonVersions) { p ->
+    Map result = new Pipenv(script: script).runWith(pythonVersions) { p ->
       numCalls++
       return p
     }
@@ -71,11 +71,11 @@ class PipenvTest extends BasePipelineTest {
 
   @Test(expected = AssertionError)
   void runWithNoScript() throws Exception {
-    new Pipenv().runWith(pythonVersions: ['2.7']) {}
+    new Pipenv().runWith(['2.7']) {}
   }
 
   @Test(expected = AssertionError)
   void runWithEmptyPythonVersions() throws Exception {
-    new Pipenv(script: script).runWith(pythonVersions: []) {}
+    new Pipenv(script: script).runWith([]) {}
   }
 }