Skip to content

Conversation

@ABMC831
Copy link

@ABMC831 ABMC831 commented Nov 15, 2024

  1. Removed deprecation from build failures - otherwise cannot build it with tools that actually respect it

  2. Changed embeded mongo provider for unit test with recent one (capable of hosting older versions as well for backwards compatibility)
    -> and set it to v 8.0

  3. Added specific unit test checking behavior of mongo, relevant to its version being after 4.4 or prior to 4.2

  4. changed unit tests influenced by this behavior.....
    the "functionality" is only available since 4.4 and in older version, it is kinda ignored....
    seems like the original code is written as if expecting the functionality being there... while unit tests are testing it being ignored

This obviously expects deployment mongo to be of version 8.0
apart from this discrepancy.... runnin unit tests, integration tests and initial manual testing of menas seems to work in either, legacy mongo 4.0 as well as mongo 8.0 (just Mongo updates needs to be done in succession on the existing database)

@ABMC831 ABMC831 requested a review from Zejnilovic November 15, 2024 10:08
@ABMC831 ABMC831 self-assigned this Nov 15, 2024
@ABMC831 ABMC831 linked an issue Nov 15, 2024 that may be closed by this pull request
@sonarqubecloud
Copy link

@benedeki
Copy link
Collaborator

  • code reviewed
  • pulled
  • built
  • run

@benedeki benedeki added the PR:no testing needed Only for PR - PR doesn't need to be tested by a tester (person) label Dec 23, 2024
@benedeki
Copy link
Collaborator

Merging on behalf of @ABMC831 who is away for longer period of time.

@benedeki benedeki merged commit 18453fa into support/2.27 Dec 23, 2024
6 of 8 checks passed
@benedeki benedeki deleted the feature/#2219-Enceladus-to-run-with-new-supported-version-of-MongoDB branch December 23, 2024 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR:no testing needed Only for PR - PR doesn't need to be tested by a tester (person)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enceladus to run with new, supported version of MongoDB

3 participants