Page MenuHomeWildfire Games

ResourceSupply unit test
ClosedPublic

Authored by fatherbushido on Sep 11 2017, 2:04 PM.

Details

Summary
  • unit tests
  • bug catch
  • diminishing return logic tests are easy but kept for later
Test Plan

bots will do the job

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

fatherbushido created this revision.Sep 11 2017, 2:04 PM
Vulcan added a subscriber: Vulcan.Sep 11 2017, 2:52 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...

http://jenkins-master:8080/job/phabricator/2014/ for more details.

Executing section Default...
Executing section Source...
Executing section JS...

binaries/data/mods/public/simulation/components/tests/test_ResourceSupply.js
|  11| »   »   »   schema·+=·"<value>"·+·"treasure."·+·res·+·"</value>";
|    | [NORMAL] ESLintBear (no-useless-concat):
|    | Unexpected string concatenation of literals.

binaries/data/mods/public/simulation/components/tests/test_ResourceSupply.js
|  35| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.
Executing section XML GUI...
Executing section Python...
Executing section Perl...

http://jenkins-master:8080/job/phabricator_lint/515/ for more details.

good job bot

This revision was automatically updated to reflect the committed changes.

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...

http://jenkins-master:8080/job/phabricator/2015/ for more details.

Executing section Default...
Executing section Source...
Executing section JS...
Executing section XML GUI...
Executing section Python...
Executing section Perl...

http://jenkins-master:8080/job/phabricator_lint/516/ for more details.

Executing section Default...
Executing section Source...
Executing section JS...
Executing section XML GUI...
Executing section Python...
Executing section Perl...

http://jenkins-master:8080/job/phabricator_lint/517/ for more details.

Build has FAILED

svn: warning: W150002: '/mnt/data/jenkins-phabricator/workspace/phabricator/binaries/data/mods/public/simulation/components/tests/test_ResourceSupply.js' is already under version control
svn: E200009: Could not add all targets because some targets are already versioned
svn: E200009: Illegal target for the requested operation
svn: E135001: Unrecognized line ending style 'native
\ No newline at end of property' for '/mnt/data/jenkins-phabricator/workspace/phabricator/binaries/data/mods/public/simulation/components/tests/test_ResourceSupply.js'

Link to build: http://jenkins-master:8080/job/phabricator/2017/
See console output for more information: http://jenkins-master:8080/job/phabricator/2017/console

elexis added a subscriber: elexis.Sep 11 2017, 6:38 PM
elexis added inline comments.
ps/trunk/binaries/data/mods/public/simulation/components/tests/test_ResourceSupply.js
76

refs #4769