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.

Sign up for GitHub

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

Jump to bottom

NMS-16460 : Rerun test in circleci. #7534

Open
wants to merge 13 commits into
base: develop
Choose a base branch
from
Open

NMS-16460 : Rerun test in circleci. #7534

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
1c410d6
NMS-16460 : Rerun test in circleci.
MusaidAli Nov 18, 2024
b359984
NMS-16460 : upgraded itext to 4.2.1
MusaidAli Nov 18, 2024
9c6bdb3
NMS-16460 : reverted back itext version.
MusaidAli Nov 18, 2024
2ec488e
NMS-16460 : Rerun tests in circleci.
MusaidAli Nov 19, 2024
3f291ed
NMS-16460 : Rerun tests in circleci.
MusaidAli Nov 19, 2024
2239147
NMS-16460 : Removed @ignore test case.
MusaidAli Nov 19, 2024
376d561
NMS-16460 : Wait for visible before click.
MusaidAli Nov 19, 2024
0b96cb9
NMS-16460 : Removed parent only view.
MusaidAli Nov 20, 2024
838cd70
NMS-16460 : Removed redundant.
MusaidAli Nov 20, 2024
2afe5e9
NMS-16460 : Reverted changes.
MusaidAli Nov 20, 2024
a4298c6
NMS-16460 : Commented out elementToBeClickable.
MusaidAli Nov 20, 2024
620505f
NMS-16460 : retesting.
MusaidAli Nov 20, 2024
098e8c1
NMS-16460 : Added @ignore.
MusaidAli Nov 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion dependencies/jasper/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -283,7 +283,7 @@
<dependency>
<groupId>com.lowagie</groupId>
<artifactId>itext</artifactId>
<version>2.1.7.js4</version>
<version>4.2.1</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Just curious, why are we updating the com.lowagie. itext version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

as we have excluded com.lowagie from jasperreports .for info
https://stackoverflow.com/questions/67579217/maven-build-failing-to-download-com-lowagieitextjar2-1-7-js6-dependency

<exclusions>
<exclusion>
<groupId>eclipse</groupId>
Expand Down
2 changes: 1 addition & 1 deletion smoke-test/src/test/java/org/opennms/smoketest/topo/GraphMLTopologyIT.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -297,7 +297,7 @@ public void verifyCanSelectNonVisibleVertex() {
}

@Test
@Ignore("Started to fail after we added pluginRepository to override the jasper-third-party url in opennms-base-assembly pom file ; see NMS-16460")
//@Ignore("Started to fail after we added pluginRepository to override the jasper-third-party url in opennms-base-assembly pom file ; see NMS-16460")
public void verifyCanSetLayerViaUrlParameter() {
adminPage(); // leave topology page to ensure the link actually works
final String namespace = "acme:markets";
Expand Down
Loading