JDO
  1. JDO
  2. JDO-277

Change wage in PartTimeEmployee instances to a small number

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: JDO 2 beta
    • Component/s: tck
    • Labels:
      None

      Description

      The test data for PartTimeEmployee includes wage of 13000 and 15000. This test data has been confused with the salary of FullTimeEmployee. I propose changing the test data for PartTimeEmployee wage from 15000 to 15 and from 13000 to 13.

      1. parttimeemployee.patch
        12 kB
        Craig L Russell
      2. parttimeemployee.patch
        2 kB
        Craig L Russell

        Activity

        Hide
        Craig L Russell added a comment -

        Committed revision 368922.

        Show
        Craig L Russell added a comment - Committed revision 368922.
        Hide
        Craig L Russell added a comment -

        This patch changes all of the test/testdata files that use PartTimeEmployee, per Michael's comments.

        Show
        Craig L Russell added a comment - This patch changes all of the test/testdata files that use PartTimeEmployee, per Michael's comments.
        Hide
        Michael Bouschen added a comment -

        The patch looks good. You might want to take a look at the other company model test data xml files, because they might define the same values for the wage. I'm not sure whether there are any tests explictly using these values. If so, they need to be updated, too.

        Show
        Michael Bouschen added a comment - The patch looks good. You might want to take a look at the other company model test data xml files, because they might define the same values for the wage. I'm not sure whether there are any tests explictly using these values. If so, they need to be updated, too.
        Hide
        Craig L Russell added a comment -

        This patch changes the test data to reasonable "wage" from 15000 to 15 and from 13000 to 13.

        No effect on test execution with this patch.

        Show
        Craig L Russell added a comment - This patch changes the test data to reasonable "wage" from 15000 to 15 and from 13000 to 13. No effect on test execution with this patch.

          People

          • Assignee:
            Craig L Russell
            Reporter:
            Craig L Russell
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development