Changes in / [38:77d4f71ee2a7:37:fe97cac0645e] in pyenvjasmine

Files:

• MANIFEST (modified) (1 diff)
• pyenvjasmine/envjasmine/lib/jasmine-3.1.0/boot.js (deleted)
• pyenvjasmine/envjasmine/lib/jasmine-3.1.0/jasmine-html.js (deleted)
• pyenvjasmine/envjasmine/lib/jasmine-3.1.0/jasmine.css (deleted)
• pyenvjasmine/envjasmine/lib/jasmine-3.1.0/jasmine.js (deleted)
• pyenvjasmine/envjasmine/lib/jasmine-jquery/jasmine3-jquery.js (deleted)
• pyenvjasmine/run-jasmine3.js (deleted)
• pyenvjasmine/runner.html (modified) (2 diffs)
• pyenvjasmine/runner.py (modified) (8 diffs)
• pyenvjasmine/runner3.html (deleted)
• pyenvjasmine/tests/sample/browser.configfile.js (modified) (1 diff)
• pyenvjasmine/tests/sample/browser.configfile_no_mocks.js (deleted)
• pyenvjasmine/tests/test_runner.py (modified) (7 diffs)

MANIFEST

@@ -1 +1 @@
 # file GENERATED by distutils, do NOT edit
 README
-setup.cfg
 setup.py
 pyenvjasmine/__init__.py

pyenvjasmine/runner.html

@@ -8 +8 @@
     <script type="text/javascript" src="file:///%(libDir)s/envjasmine/lib/jasmine/jasmine.js"></script>
     <script type="text/javascript" src="file:///%(libDir)s/envjasmine/lib/jasmine/jasmine-html.js"></script>
+    <script type="text/javascript" src="file:///%(libDir)s/envjasmine/lib/jasmine-ajax/mock-ajax.js"></script>
+    <script type="text/javascript" src="file:///%(libDir)s/envjasmine/lib/jasmine-ajax/spec-helper.js"></script>
+    <script type="text/javascript" src="file:///%(libDir)s/envjasmine/lib/jasmine-jquery/jasmine-jquery.js"></script>
 
     <script type="text/javascript" src="browser.runner.js"></script>
@@ -40 +43 @@
       EnvJasmine.jsDir = "";
       EnvJasmine.testDir = "%(testDir)s";
-      EnvJasmine.envJasmineLibDir = 'file:///%(libDir)s/envjasmine/lib';
 
       EnvJasmine.load = function (path) {

pyenvjasmine/runner.py

@@ -89 +89 @@
 
     watchdog.cancel()  # if it's still waiting to run
+
     # if it timed out, success is False
     success = (not kill_check.isSet()) and p.returncode >= 0
@@ -94 +95 @@
     return (success, ''.join(stdout_l), stderr)
 
+
 class Runner(object):
     """
@@ -103 +105 @@
 
     def __init__(self, rootdir=None, testdir=None, configfile=None,
-                 browser_configfile=None, testing_enviroment='phantomjs'):
+                 browser_configfile=None):
         """
         Set up paths, by default everything is
@@ -125 +127 @@
         self.configfile = configfile
         self.browser_configfile = browser_configfile
-        self.runner_js = os.path.join(here, 'run-jasmine3.js')
-        self.testing_enviroment = testing_enviroment
+        self.runner_html = os.path.join(here, 'runner.html')
+
+    def run(self, spec=None, timeout=None):
+        """
+        Run the js tests with envjasmine, return success (true/false) and
+        the captured stdout data
+
+        spec: (relative) path to a spec file (run only that spec)
+        timeout: Set it to a given number of seconds and the process running
+                 the js tests will be killed passed that time
+        """
         environment = get_environment()
         rhino_path = os.path.join(self.rootdir, 'lib', 'rhino', 'js.jar')
@@ -132 +143 @@
         rootdir_param = '--rootDir=%s' % self.rootdir
         testdir_param = '--testDir=%s' % self.testdir
-
-        # using a dictionary to parameterize the different engines
-        self.envs = {
-            'phantomjs' : {
-                'command' : [
-                    'phantomjs',
-                    '--debug=true',
-                    self.runner_js,
-                    'browser.runner.html'
-                ],
-                'runner_html' : 'runner3.html',
-                'failed_mark' : 'FAILED',
-                'success_mark' : '0 failures'
-            },
-            'rhino' : {
-                'command' : [
-                    'java',
-                    '-Duser.timezone=US/Eastern',
-                    '-Dfile.encoding=utf-8',
-                    '-jar',
-                    rhino_path,
-                    envjasmine_js_path,
-                    '--disableColor',
-                    environment,
-                    rootdir_param,
-                    testdir_param
-                ],
-                'runner_html' : 'runner.html',
-                'failed_mark' : 'FAILED',
-                'success_mark' : 'Failed: 0',
-                'command_params_func' : self.command_params_rhino
-            }
-        }
-        self.runner_html = os.path.join(here,
-                              self.envs[self.testing_enviroment]['runner_html'])
-
-    def command_params_rhino(self, command):
-        """
-        Function specific to Rhino to add eventual arguments to the command line.
-        The function is referenced in the dictionary "envs" with key
-        "command_params_func", and for every engine that needs the same type of
-        manipulation there will be a similar function with relative reference in
-        the dictionary
-        """
-        if self.configfile and os.path.exists(self.configfile):
-           command.append('--configFile=%s' % self.configfile)
-        return command
-
-    def run(self, spec=None, timeout=None):
-        """
-        Run the js tests with envjasmine, return success (true/false) and
-        the captured stdout data
-
-        spec: (relative) path to a spec file (run only that spec)
-        timeout: Set it to a given number of seconds and the process running
-                 the js tests will be killed passed that time
-        """
         if self.browser_configfile and os.path.exists(self.browser_configfile):
             self.write_browser_htmlfile()
-        command = self.envs[self.testing_enviroment]['command']
-        # Add eventual other parameters to the command by calling a
-        # function specific for the selected engine (rhino, phantomjs, etc.)
-        if 'command_params_func' in self.envs[self.testing_enviroment]:
-            command = self.envs[self.testing_enviroment]['command_params_func'](command)
+
+        command = [
+            'java',
+            '-Duser.timezone=US/Eastern',
+            '-Dfile.encoding=utf-8',
+            '-jar',
+            rhino_path,
+            envjasmine_js_path,
+            '--disableColor',
+            environment,
+            rootdir_param,
+            testdir_param
+            ]
+
+        if self.configfile and os.path.exists(self.configfile):
+            command.append('--configFile=%s' % self.configfile)
+
         # if we were asked to test only some of the spec files,
         # addd them to the command line:
@@ -208 +174 @@
         stderr = subprocess.PIPE
         input_data = ''
+
         success, stdout, stderr = run_popen_with_timeout(
             command, timeout, input_data, stdin, stdout, stderr
@@ -219 +186 @@
 
     def did_test_pass(self, stdout):
-        if self.envs[self.testing_enviroment]['failed_mark'] in stdout:
+        if 'FAILED' in stdout:
             # it can happen that a test fails because of some timing issues
             # (timer error). In such case it may happen that the test does
@@ -231 +198 @@
         # tests passing ok
         for line in stdout.splitlines():
-            if self.envs[self.testing_enviroment]['success_mark'] in line:
-                return True
+            if 'Failed' in line:
+                failed = line.split(':')[1].strip()
+                return failed == '0'
         return False
 

pyenvjasmine/tests/sample/browser.configfile.js

@@ -6 +6 @@
 EnvJasmine.jsDir = EnvJasmine.testDir + "/code/";
 EnvJasmine.loadGlobal(EnvJasmine.includeDir + "jquery-1.4.4.js");
-EnvJasmine.loadGlobal(EnvJasmine.testDir + '/code/demo.js');
-EnvJasmine.loadGlobal(EnvJasmine.testDir + '/mocks/demo.mock.js');
-EnvJasmine.loadGlobal(EnvJasmine.testDir + '/tests/specs/test_demo.spec.js');

pyenvjasmine/tests/test_runner.py

@@ -5 +5 @@
 
 
-class TestsRunnerRhino(object):
-
-    """
-    Run the full tests using the old rhino+jasmine1 env
-    """
+class TestsRunner(object):
 
     def test_runner_defaults(self):
@@ -15 +11 @@
         Test the runner, using default values (which wil run the demo specs)
         """
-        jstests = Runner(testing_enviroment='rhino')
+        jstests = Runner()
         success, stdout = jstests.run()
         assert success
@@ -30 +26 @@
         envjasmine_dir = os.path.join(os.path.dirname(here), 'envjasmine')
         jstests = Runner(
-            testing_enviroment='rhino',
             rootdir=envjasmine_dir,
             testdir=sample,
@@ -52 +47 @@
         envjasmine_dir = os.path.join(os.path.dirname(here), 'envjasmine')
         jstests = Runner(
-            testing_enviroment='rhino',
             rootdir=envjasmine_dir,
             testdir=sample,
@@ -74 +68 @@
         envjasmine_dir = os.path.join(os.path.dirname(here), 'envjasmine')
         jstests = Runner(
-            testing_enviroment='rhino',
             rootdir=envjasmine_dir,
             testdir=sample,
@@ -104 +97 @@
         # but we want to test also the case when a test failed
         # and it does not appear in the "Failed:" report
-        jstests = Runner(testing_enviroment='rhino')
+        jstests = Runner()
         success = jstests.did_test_pass('')
         assert not success
@@ -127 +120 @@
         success = jstests.did_test_pass('Failed: something-not-a-number')
         assert not success
-
-class TestsRunnerPhantomjs(object):
-
-    """
-    Run the full tests using the phantom+jasmine3 env
-    """
-
-    def test_write_browser_htmlfile_markup_is_correct(self):
-        """
-        Test the created markup
-        The config file doesn't contain the mock files so jasmine tests
-        are expected to fail
-        """
-        here = os.path.dirname(__file__)
-        sample = os.path.join(here, 'sample')
-        browser_conf_file = os.path.join(sample, 'browser.configfile_no_mocks.js')
-        envjasmine_dir = os.path.join(os.path.dirname(here), 'envjasmine')
-        jstests = Runner(
-            testing_enviroment='phantomjs',
-            rootdir=envjasmine_dir,
-            testdir=sample,
-            browser_configfile=browser_conf_file
-        )
-        expected = jstests.create_testRunnerHtml()
-        jstests.write_browser_htmlfile()
-        success, stdout = jstests.run()
-        assert not success
-        assert '2 test(s) FAILED:' in stdout
-        with open("browser.runner.html",'r') as file:
-            actual = file.read()
-            assert expected == actual
-
-    def test_runner_with_browser_configfile(self):
-        """
-        Test the runner, giving it some parameters incl the browser config file
-        The config file contains the mock files so jasmine tests are expected
-        to pass
-        """
-        here = os.path.dirname(__file__)
-        sample = os.path.join(here, 'sample')
-        browser_conf_file = os.path.join(sample, 'browser.configfile.js')
-        envjasmine_dir = os.path.join(os.path.dirname(here), 'envjasmine')
-        jstests = Runner(
-            testing_enviroment='phantomjs',
-            rootdir=envjasmine_dir,
-            testdir=sample,
-            browser_configfile=browser_conf_file
-        )
-        success, stdout = jstests.run()
-        assert success
-        assert '0 failures' in stdout
-
-    def test_did_test_pass(self):
-        # there is some coverage done with the previous tests,
-        # but we want to test also the case when a test failed
-        # and it does not appear in the "Failed:" report
-        here = os.path.dirname(__file__)
-        sample = os.path.join(here, 'sample')
-        browser_conf_file = os.path.join(sample, 'browser.configfile.js')
-
-        jstests = Runner(
-            testing_enviroment='phantomjs',
-            browser_configfile=browser_conf_file
-        )
-        success = jstests.did_test_pass('')
-        assert not success
-        success = jstests.did_test_pass('some random data '*50)
-        assert not success
-        success = jstests.did_test_pass('some data FAILED some more data')
-        assert not success
-        success = jstests.did_test_pass('some data FAILEDsome more data')
-        assert not success
-        success = jstests.did_test_pass('0 failures')
-        assert success
-        success = jstests.did_test_pass('0 failures FAILED')
-        assert not success
-        success = jstests.did_test_pass('0Failed: failures')
-        assert not success
-        success = jstests.did_test_pass('1 test(s) FAILED:')
-        assert not success
-        success = jstests.did_test_pass('-11 test(s) FAILED:')
-        assert not success
-        success = jstests.did_test_pass('something-not-a-number test(s) FAILED:')
-        assert not success