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

GitHub noreply at github.com
Thu Jul 18 01:26:33 UTC 2013


  Branch: refs/heads/integration/master
  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.


  Commit: df00dc1cab4e993a5f0d98cf84faacb8f60f6381
      https://github.com/zanata/zanata-server/commit/df00dc1cab4e993a5f0d98cf84faacb8f60f6381
  Author: Patrick Huang <pahuang 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:
  -----------
  Merge pull request #53 from zanata/for-loops-in-tests-are-bad

Remove for loops and conditions in tests.


Compare: https://github.com/zanata/zanata-server/compare/1fc5a53d7d2a...df00dc1cab4e


More information about the zanata-commits mailing list