Skip to content
This repository was archived by the owner on Jan 30, 2020. It is now read-only.

Conversation

@dongsupark
Copy link

Introduce a new test for the fleetctl submit command,
not only for simple units, but also for template units.

NOTE: this PR depends on #1466, which is still pending.

@dongsupark
Copy link
Author

I updated the tree. Changes are:

  • in doSubmitUnits(), check whether each input unit exists in the submitted unit.
  • remove a redundant flag setting sharedFlags.NoBlock.

@tixxdz
Copy link
Contributor

tixxdz commented Mar 8, 2016

lgtm, @jonboulle #1466 then this one. Thanks!

@jonboulle
Copy link
Contributor

Please rebase

@dongsupark dongsupark force-pushed the dongsu/fleetctl-submit-test branch from 23bc63f to 2647b45 Compare March 8, 2016 19:45
@dongsupark
Copy link
Author

@jonboulle rebased.

0,
},
{
"submit same unit from an available template",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might as well add some cases that mix non-available/available, and same units, etc.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jonboulle Thanks, I'll do.

Introduce a new test for the fleetctl submit command,
not only for simple units, but also for template units.
@dongsupark dongsupark force-pushed the dongsu/fleetctl-submit-test branch from 2647b45 to b6cc520 Compare March 9, 2016 14:18
@dongsupark
Copy link
Author

Rebased and updated.
As @jonboulle suggested, the following tests were added:

  • submit same unit multiple times
  • submit same unit with non-available units
  • submit units from available and non-available templates
  • submit same unit from an available template with units from non-available templates

@jonboulle
Copy link
Contributor

LGTM, thnaks

jonboulle added a commit that referenced this pull request Mar 9, 2016
fleetctl:test: add a new test for submit command
@jonboulle jonboulle merged commit 49ca5fb into coreos:master Mar 9, 2016
@dongsupark dongsupark deleted the dongsu/fleetctl-submit-test branch April 5, 2016 09:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants