Summary

Add unit tests for Travis CI.

Review Request #8937 — Created May 10, 2017 and submitted July 14, 2017, 4:21 p.m.

Owner

Repository

rbintegrations

Branch

master

Bugs

Depends On

Commit

610defd...

Groups

People

Description

This change adds a test suite for the Travis CI integration.

Testing Done

Ran unit tests.

Issues

Description	From	Last Updated
Summary's a bit weird.	chipx86	July 8, 2017, 2:02 p.m.
F841 local variable 'config' is assigned to but never used	reviewbot	May 10, 2017, 2:06 p.m.
E303 too many blank lines (2)	reviewbot	May 10, 2017, 2:06 p.m.
Description doesn't seem to match the test name very well, and is repeated in the following test.	chipx86	July 8, 2017, 2:03 p.m.
, optional	chipx86	July 8, 2017, 2:03 p.m.
, optional	chipx86	July 8, 2017, 2:03 p.m.

flake8 failed.

JSHint passed.

rbintegrations/travisci/tests.py (Diff revision 1)
The issue has been resolved. Show all issues
```
F841 local variable 'config' is assigned to but never used
```
rbintegrations/travisci/tests.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E303 too many blank lines (2)
```

Commit:

405f11c4ab96c79c8d108ac0e42c3b15efccd694

a686795a4b6a6e4a4fb372c9996c34ef9cca6bfc

Diff:

flake8 passed.

JSHint passed.

rbintegrations/travisci/tests.py (Diff revision 2)

The issue has been resolved. Show all issues

Description doesn't seem to match the test name very well, and is repeated in the following test.

rbintegrations/travisci/tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
, optional
```
rbintegrations/travisci/tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
, optional
```