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 chart core tests #685 #686

Merged
merged 12 commits into from
Nov 13, 2021
Merged

Fix chart core tests #685 #686

merged 12 commits into from
Nov 13, 2021

Conversation

wimjongman
Copy link
Contributor

Added fragment to provide dependency [1]
Fixed some Locale dependent tests

[1]
The chart core tests were failing due to an incompatible
java/osgi construct in org.apache.batik.transcoder.

The transcoder "bundle" is doing a Class.forname(String)
on a class that is not listed as a dependency.

This happens in class
org.apache.batik.transcoder.image.PNGTranscoder#getWriteAdapter
which is looking for
"org.apache.batik.ext.awt.image.codec.png.PNGTranscoderInternalCodecWriteAdapter"

To fool this "bundle", a fragment bundle is introduced with
o.a.b.transcoder as the master plugin that depends on the
"org.apache.batik.codec" bundle and in this way provides the dependency.

@wimjongman wimjongman linked an issue Nov 2, 2021 that may be closed by this pull request
Added some java documentation
Added fragment to provide dependency [1]
Fixed some Locale dependent tests


[1]
The chart core tests were failing due to an incompatible
java/osgi construct in org.apache.batik.transcoder.

The transcoder "bundle" is doing a Class.forname(String)
on a class that is not listed as a dependency.

This happens in class
org.apache.batik.transcoder.image.PNGTranscoder#getWriteAdapter
which is looking for
"org.apache.batik.ext.awt.image.codec.png.PNGTranscoderInternalCodecWriteAdapter"

To fool this "bundle", a fragment bundle is introduced with
o.a.b.transcoder as the master plugin that depends on the
"org.apache.batik.codec" bundle and in this way provides the dependency.
Added fragment to provide dependency [1]
Fixed some Locale dependent tests


[1]
The chart core tests were failing due to an incompatible
java/osgi construct in org.apache.batik.transcoder.

The transcoder "bundle" is doing a Class.forname(String)
on a class that is not listed as a dependency.

This happens in class
org.apache.batik.transcoder.image.PNGTranscoder#getWriteAdapter
which is looking for
"org.apache.batik.ext.awt.image.codec.png.PNGTranscoderInternalCodecWriteAdapter"

To fool this "bundle", a fragment bundle is introduced with
o.a.b.transcoder as the master plugin that depends on the
"org.apache.batik.codec" bundle and in this way provides the dependency.
Added fragment to provide dependency [1]
Fixed some Locale dependent tests


[1]
The chart core tests were failing due to an incompatible
java/osgi construct in org.apache.batik.transcoder.

The transcoder "bundle" is doing a Class.forname(String)
on a class that is not listed as a dependency.

This happens in class
org.apache.batik.transcoder.image.PNGTranscoder#getWriteAdapter
which is looking for
"org.apache.batik.ext.awt.image.codec.png.PNGTranscoderInternalCodecWriteAdapter"

To fool this "bundle", a fragment bundle is introduced with
o.a.b.transcoder as the master plugin that depends on the
"org.apache.batik.codec" bundle and in this way provides the dependency.

Also added batik.w3c.dom as pkg dependency to the same
Added fragment to provide dependency [1]
Fixed some Locale dependent tests


[1]
The chart core tests were failing due to an incompatible
java/osgi construct in org.apache.batik.transcoder.

The transcoder "bundle" is doing a Class.forname(String)
on a class that is not listed as a dependency.

This happens in class
org.apache.batik.transcoder.image.PNGTranscoder#getWriteAdapter
which is looking for
"org.apache.batik.ext.awt.image.codec.png.PNGTranscoderInternalCodecWriteAdapter"

To fool this "bundle", a fragment bundle is introduced with
o.a.b.transcoder as the master plugin that depends on the
"org.apache.batik.codec" bundle and in this way provides the dependency.

An identical trick was pulled adding batik.w3c.dom as pkg dependency
to org.apache.batik.dom
Added o.a.batik.dom.birt.ext to birt feature

This is a fragment extending batik.dom so that
it can see org.apache.batik.w3c.dom

It solves the following dependency issue:

java.lang.NoClassDefFoundError:
org/apache/batik/w3c/dom/ElementTraversal
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Caused by: java.lang.ClassNotFoundException:
org.apache.batik.w3c.dom.ElementTraversal cannot be found by
org.apache.batik.dom_1.14.0.v20210324-0332
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Added o.a.batik.dom.birt.ext to birt feature

Added batik.ext 1.14 to target

This is a fragment extending batik.dom so that
it can see org.apache.batik.w3c.dom

It solves the following dependency issue:

java.lang.NoClassDefFoundError:
org/apache/batik/w3c/dom/ElementTraversal
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Caused by: java.lang.ClassNotFoundException:
org.apache.batik.w3c.dom.ElementTraversal cannot be found by
org.apache.batik.dom_1.14.0.v20210324-0332
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Added o.a.batik.dom.birt.ext to birt feature

Added batik.ext 1.14 to target

This is a fragment extending batik.dom so that
it can see org.apache.batik.w3c.dom

It solves the following dependency issue:

java.lang.NoClassDefFoundError:
org/apache/batik/w3c/dom/ElementTraversal
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Caused by: java.lang.ClassNotFoundException:
org.apache.batik.w3c.dom.ElementTraversal cannot be found by
org.apache.batik.dom_1.14.0.v20210324-0332
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Added o.a.batik.dom.birt.ext to birt feature

Added batik.ext 1.14 to target

This is a fragment extending batik.dom so that
it can see org.apache.batik.w3c.dom

It solves the following dependency issue:

java.lang.NoClassDefFoundError:
org/apache/batik/w3c/dom/ElementTraversal
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Caused by: java.lang.ClassNotFoundException:
org.apache.batik.w3c.dom.ElementTraversal cannot be found by
org.apache.batik.dom_1.14.0.v20210324-0332
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Added o.a.batik.dom.birt.ext to birt feature

Added batik.ext 1.14 to target

This is a fragment extending batik.dom so that
it can see org.apache.batik.w3c.dom

It solves the following dependency issue:

java.lang.NoClassDefFoundError:
org/apache/batik/w3c/dom/ElementTraversal
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Caused by: java.lang.ClassNotFoundException:
org.apache.batik.w3c.dom.ElementTraversal cannot be found by
org.apache.batik.dom_1.14.0.v20210324-0332
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Added o.a.batik.dom.birt.ext to birt feature

Added batik.ext 1.14 to target

Disable the test so that it can at least run locally.

This is a fragment extending batik.dom so that
it can see org.apache.batik.w3c.dom

It solves the following dependency issue:

java.lang.NoClassDefFoundError:
org/apache/batik/w3c/dom/ElementTraversal
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Caused by: java.lang.ClassNotFoundException:
org.apache.batik.w3c.dom.ElementTraversal cannot be found by
org.apache.batik.dom_1.14.0.v20210324-0332
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Added o.a.batik.dom.birt.ext to birt feature

Added batik.ext 1.14 to target

Disable the test so that it can at least run locally.

This is a fragment extending batik.dom so that
it can see org.apache.batik.w3c.dom

It solves the following dependency issue:

java.lang.NoClassDefFoundError:
org/apache/batik/w3c/dom/ElementTraversal
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
Caused by: java.lang.ClassNotFoundException:
org.apache.batik.w3c.dom.ElementTraversal cannot be found by
org.apache.batik.dom_1.14.0.v20210324-0332
	at org.eclipse.birt.chart.tests.device.render.ImageOutputBaseTest.runTest(ImageOutputBaseTest.java:100)
@wimjongman
Copy link
Contributor Author

Alexander, I can't get it to work in the CI build. However, with this patch, we can at least run the test locally.

@wimjongman wimjongman removed the request for review from ruspl-afed November 13, 2021 14:02
@wimjongman wimjongman merged commit ca854df into master Nov 13, 2021
@wimjongman wimjongman deleted the Fix_chart_core_tests_#685 branch November 13, 2021 14:02
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.

Fix chart core tests
1 participant