Page MenuHomeWildfire Games
Differential D240

Unit tests for JS Pack component
ClosedPublic
Actions

Authored by fatherbushido on Mar 20 2017, 10:16 PM.

Details

Reviewers
wraitii
Itms
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Commits
rP19350: Unit tests for JS Pack component
Summary

Unit tests.

Test Plan

Irrelevant.

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.Mar 20 2017, 10:16 PM
Vulcan added a subscriber: Vulcan.Mar 20 2017, 11:01 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/557/ for more details.

Harbormaster completed remote builds in B821: Diff 870.Mar 20 2017, 11:01 PM
Sandarac added a subscriber: Sandarac.Mar 21 2017, 8:29 AM

I just noticed a few missing semicolons.

binaries/data/mods/public/simulation/components/tests/test_Pack.js
80 ↗(On Diff #870)

Missing semicolon.

83 ↗(On Diff #870)

Missing semicolon.

105 ↗(On Diff #870)

Missing semicolon.

113 ↗(On Diff #870)

Missing semicolon.

116 ↗(On Diff #870)

Missing semicolon.

fatherbushido added a comment.Mar 21 2017, 10:06 AM

Thanks!

fatherbushido updated this revision to Diff 874.Mar 21 2017, 2:34 PM

Missing semicolon noticed by Sandarac

Vulcan added a comment.Mar 21 2017, 3:20 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/559/ for more details.

Harbormaster completed remote builds in B825: Diff 874.Mar 21 2017, 3:20 PM
fatherbushido updated this revision to Diff 885.Mar 22 2017, 1:27 PM

Actually test unpack completion.

Harbormaster completed remote builds in B833: Diff 885.Mar 22 2017, 1:27 PM
Owners added subscribers: Restricted Owners Package, Restricted Owners Package, Restricted Owners Package.Mar 22 2017, 1:27 PM
Vulcan added a comment.Mar 22 2017, 2:12 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/566/ for more details.

Harbormaster completed remote builds in B833: Diff 885.Mar 22 2017, 2:13 PM
fatherbushido updated this revision to Diff 886.Mar 22 2017, 2:55 PM

eol-style=native

fatherbushido updated this revision to Diff 887.Mar 22 2017, 3:07 PM

Remove unrelated files

Vulcan added a comment.Mar 22 2017, 3:40 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/567/ for more details.

Harbormaster completed remote builds in B834: Diff 886.Mar 22 2017, 3:40 PM
Vulcan added a comment.Mar 22 2017, 4:25 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/568/ for more details.

Harbormaster completed remote builds in B835: Diff 887.Mar 22 2017, 4:25 PM
fatherbushido added a reviewer: wraitii.Mar 25 2017, 9:46 AM
fatherbushido updated this revision to Diff 966.Mar 27 2017, 1:28 PM

Explain the 250 const as suggested by wraitii

wraitii accepted this revision.Mar 27 2017, 1:32 PM
This revision is now accepted and ready to land.Mar 27 2017, 1:32 PM
fatherbushido updated this revision to Diff 967.Mar 27 2017, 1:38 PM

Add native eol-style property

Vulcan added a comment.Mar 27 2017, 2:14 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/615/ for more details.

Harbormaster completed remote builds in B908: Diff 966.Mar 27 2017, 2:14 PM
Closed by commit rP19350: Unit tests for JS Pack component (authored by fatherbushido). · Explain WhyMar 27 2017, 2:41 PM
This revision was automatically updated to reflect the committed changes.
Vulcan added a comment.Mar 27 2017, 3:01 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (305 tests).................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/616/ for more details.

Harbormaster completed remote builds in B909: Diff 967.Mar 27 2017, 3:01 PM

Revision Contents
Changeset List

PathSize
ps/
trunk/
binaries/
data/
mods/
public/
simulation/
components/
tests/
158 lines

Diff 970

View Options

ps/trunk/binaries/data/mods/public/simulation/components/tests/test_Pack.js

Loading...
Wildfire Games · Phabricator