feat: Add desired tasks

[romankydybets]

I would like to ask for help in the PR, as I can't fix 1 test which is failing.

Issue #499

Add support for DesiredCount
it would be great to have DesiredCount support during deployment.
https://docs.aws.amazon.com/cli/latest/reference/ecs/update-service.html
--desired-count

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[romankydybets]

@clareliguori could i ask for help to solve issue with test?

[KollaAdithya]

Hello @romankydybets !

Thanks for the PR! Is there some error you are getting when running the tests? can you paste the error over here?

[romankydybets]

@KollaAdithya
this is what i have, honestly i try to debug on my own, but unfortunately can't achieve this

  ● Deploy to ECS › does not wait for a CodeDeploy deployment, parses JSON appspec file

    expect(jest.fn()).toHaveBeenCalledTimes(expected)

    Expected number of calls: 0
    Received number of calls: 1

      767 |
      768 |         await run();
    > 769 |         expect(core.setFailed).toHaveBeenCalledTimes(0);
          |                                ^
      770 |
      771 |         expect(mockEcsRegisterTaskDef).toHaveBeenNthCalledWith(1, { family: 'task-def-family'});
      772 |         expect(core.setOutput).toHaveBeenNthCalledWith(1, 'task-definition-arn', 'task:def:arn');

      at Object.toHaveBeenCalledTimes (index.test.js:769:32)

[KollaAdithya]

Hi @romankydybets !
In order to fix the unit test cases you can just modify the test case with below ⬇️ . I already ran testcases with the modification locally, all the testcases are getting passed.

test('does not wait for a CodeDeploy deployment, parses JSON appspec file', async () => {
        core.getInput = jest
            .fn()
            .mockReturnValueOnce('task-definition.json') // task-definition
            .mockReturnValueOnce('service-456')          // service
            .mockReturnValueOnce('cluster-789')          // cluster
            .mockReturnValueOnce('false')                // wait-for-service-stability
            .mockReturnValueOnce('')                     // wait-for-minutes
            .mockReturnValueOnce('')                     // force-new-deployment
            .mockReturnValueOnce('')                     // desired count
            .mockReturnValueOnce('/hello/appspec.json')  // codedeploy-appspec
            .mockReturnValueOnce('MyApplication')        // codedeploy-application
            .mockReturnValueOnce('MyDeploymentGroup');    // codedeploy-deployment-group

[romankydybets]

thank you @KollaAdithya, did a fix, please take a look.

[romankydybets]

Is there any updates on this?

[marns93]

Any updates?

[romankydybets]

@clareliguori could you please review?

stale review

[marns93]

@KollaAdithya @iamhopaul123 Is it possible to publish a new release?

[mendhak]

Please could this be released, it's very useful!