test: Run integration tests against v2 as well

[nightscape]

[nightscape]

@CodiumAI-Agent

[CodiumAI-Agent]

PR Analysis

• 🎯 Main theme: Running integration tests against v2
• 🔍 Description and title: No
• 📌 Type of PR: Tests
• 🧪 Relevant tests added: Yes
• ✨ Minimal and focused: Yes, the PR is focused on updating the test suite to run against v2 of the application.
• 🔒 Security concerns: No, the changes made in this PR are related to testing and do not introduce any new security concerns.

PR Feedback

• 💡 General PR suggestions: The PR lacks a description which is important for understanding the context and the reason for the changes. Please add a detailed description explaining the need for these changes and how they affect the overall functionality of the application.

• 🤖 Code suggestions:

  • relevant file: IntegrationSuite.scala suggestion content: Consider using constants for repeated string values such as 'excel' and 'com.crealytics.spark.excel'. This will make it easier to manage and update these values in the future. [medium]

  • relevant file: IntegrationSuite.scala suggestion content: The 'runTests' function has been modified to include an 'implementation' parameter. It would be beneficial to update the function documentation to reflect this change and explain the purpose and expected values of this new parameter. [medium]

How to use

Tag me in a comment '@CodiumAI-Agent' to ask for a new review after you update the PR. You can also tag me and ask any question, for example '@CodiumAI-Agent is the PR ready for merge?'