[zanata/zanata-server] cb1ede: Remove for loops and conditions in tests.

GitHub noreply at github.com
Thu Jul 18 00:32:06 UTC 2013


  Branch: refs/heads/for-loops-in-tests-are-bad
  Home:   https://github.com/zanata/zanata-server
  Commit: cb1ede979a9ff43eb1ce33a976f2ae67fab6ede9
      https://github.com/zanata/zanata-server/commit/cb1ede979a9ff43eb1ce33a976f2ae67fab6ede9
  Author: Damian Jansen <djansen at redhat.com>
  Date:   2013-07-17 (Wed, 17 Jul 2013)

  Changed paths:
    M functional-test/src/test/java/org/zanata/feature/startNewProject/AddLanguageTest.java
    M functional-test/src/test/java/org/zanata/feature/versionGroup/VersionGroupFullTest.java
    A functional-test/src/test/java/org/zanata/feature/versionGroup/VersionGroupIDValidationTest.java

  Log Message:
  -----------
  Remove for loops and conditions in tests.

For loops and if conditions are sub-optimal in tests, for both
stability and readability. Removed and replaced with Theory run
features.
Though this adds some time to the detailed tests, it's more correct.


  Commit: cbd2656b15a26ba3ef46b181bb7a4a0e9f57e439
      https://github.com/zanata/zanata-server/commit/cbd2656b15a26ba3ef46b181bb7a4a0e9f57e439
  Author: Damian Jansen <djansen at redhat.com>
  Date:   2013-07-17 (Wed, 17 Jul 2013)

  Changed paths:
    M functional-test/src/test/java/org/zanata/feature/startNewProject/AddLanguageTest.java

  Log Message:
  -----------
  Fix AddLanguageTest.java

Broken in previous commit.


Compare: https://github.com/zanata/zanata-server/compare/1640bbc9e2c9...cbd2656b15a2


More information about the zanata-commits mailing list