Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(integ): special case for version 10.4.0.13 Repository installer #1464

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

rondeau-aws
Copy link
Contributor

Problem

The Deadline 10.4.0.13 Repository installer has a mismatched version number, such that the version in the installer filename (10.4.0.13) is different from the version embedded in the build (10.4.0.12). This causes errors in our deadline_01_repository integration test such as:

  ● Deadline Repository tests (RFDK-created DB and EFS) › EFS tests › DL-1-3: repository.ini version matches Deadline installer
    expect(received).toEqual(expected) // deep equality
    Expected: StringMatching /\[DeadlineRepository\]\nVersion=10\.4.0.13/
    Received: "[DeadlineRepository]
    Version=10.4.0.12
    VersionInformation=4a229067b
    DeploymentVersion=3
    [DeadlineIntegration]
    10.4.0.13
    VersionInformation=4a229067b
    "
      193 |       }
      194 |       const regex = new RegExp('\\[DeadlineRepository\\]\nVersion=' + expectedVersion.replace('.', '\\.'));
    > 195 |       expect(output).toEqual(expect.stringMatching(regex));
          |                      ^
      196 |     });
      197 |   });
      198 |

Solution

Add a special case so the test will expect to find Version=10.4.0.12 in the Deadline 10.4.0.13 installer.

Testing

Ran the deadline_01_repository integration test for Deadline 10.4.0.13 and confirmed that this error no longer appears.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@rondeau-aws rondeau-aws merged commit b837588 into aws:mainline Feb 11, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants