Add tests (#45)

* add worker main tests
* add manage test
* fix timezone warnings
* add WorkflowDetailForm tests

Author: aclowes

frontend/src/tests/WorkflowDetailForm.test.js

@@ -3,6 +3,7 @@ import ReactTestRenderer from 'react-test-renderer';
 
 import WorkflowDetailForm from '../WorkflowDetailForm';
 import {mockAPI} from './mocks'
+import API from "../API";
 
 it('WorkflowDetailForm editable', () => {
   mockAPI([{versions: [1]}]);
@@ -21,3 +22,46 @@ it('WorkflowDetailForm editable', () => {
   let tree = component.toJSON();
   expect(tree).toMatchSnapshot();
 });
+
+it('WorkflowDetailForm componentWillReceiveProps', () => {
+  mockAPI([{versions: []}]);
+  const workflow = {
+      "id": 1,
+      "name": "Simple Workflow Example",
+      "name_id": 1,
+      "version": 1,
+      "schedule_active": false,
+      "schedule": null,
+      "next_run": null,
+      "parameters": {"MY_OBJECT_ID": "1", "SOME_SETTING": "false"}
+    };
+  const component = ReactTestRenderer.create(<WorkflowDetailForm workflow={workflow}/>);
+  const instance = component.getInstance();
+  instance.setState = jest.fn();
+  // same but different object
+  instance.componentWillReceiveProps({workflow: {...workflow}});
+  expect(instance.setState.mock.calls[0][0].parameters).toEqual('MY_OBJECT_ID=1\nSOME_SETTING=false');
+});
+
+it('WorkflowDetailForm submit', () => {
+  mockAPI([{versions: []}]);
+  const workflow = {
+      "id": 1,
+      "name": "Simple Workflow Example",
+      "name_id": 1,
+      "version": 1,
+      "schedule_active": false,
+      "schedule": null,
+      "next_run": null,
+      "parameters": {"MY_OBJECT_ID": "1", "SOME_SETTING": "false"}
+    };
+  const component = ReactTestRenderer.create(<WorkflowDetailForm workflow={workflow}/>);
+  const instance = component.getInstance();
+  instance.toggleEditable();
+  const event = {preventDefault: jest.fn()};
+  API.patch = jest.fn();
+  instance.handleSubmit(event);
+  expect(event.preventDefault).toBeCalled();
+  // this does a roundtrip through formValues:
+  expect(API.patch.mock.calls[0][1].parameters).toEqual(workflow.parameters);
+});

yawn/management/tests/test_manage.py

@@ -0,0 +1,9 @@
+from unittest import mock
+
+from yawn import manage
+
+
+@mock.patch.object(manage, 'execute_from_command_line')
+def test_manage(mock_execute):
+    manage.main()
+    assert mock_execute.called

yawn/utilities/cron.py

@@ -33,7 +33,8 @@ def next_run(self, current_time: datetime.datetime) -> datetime.datetime:
         # if next run is next day, get smallest hour, smallest minute
         # if next run is today, future hour, get smallest minute
         # if next run is today, this hour, get next greatest minute
-        next_run = datetime.datetime(current_time.year, current_time.month, current_time.day)
+        next_run = datetime.datetime(current_time.year, current_time.month, current_time.day,
+                                     tzinfo=current_time.tzinfo)
         weekday = current_time.isoweekday()
         weekday = 0 if weekday == 7 else weekday  # Move Sunday to day 0
         if weekday in self.weekdays:

yawn/workflow/tests/test_models.py

@@ -9,6 +9,7 @@
 import datetime
 from unittest import mock
 
+import pytz
 from django.utils import timezone
 
 from yawn.task.models import Template, Task
@@ -39,7 +40,7 @@ def test_first_ready():
 
 @mock.patch('yawn.workflow.models.Crontab')
 def test_next_run(mock_cron):
-    next_run = timezone.datetime(2011, 1, 1)
+    next_run = timezone.datetime(2011, 1, 1, tzinfo=pytz.UTC)
     mock_cron().next_run.return_value = next_run
 
     name = WorkflowName.objects.create(name='workflow1')