unit test bootstrap no securitymanager#9284
Conversation
| assertEquals("Five dependencies: " + s, 5, s.size()); | ||
| assertEquals(new HashSet<>(Arrays.asList("init", "all-openide.dialogs", "all-openide.util", "all-openide.util.lookup", "all-api.annotations.common")), s); |
There was a problem hiding this comment.
here dependencies count are not 5 but 2
| "dname='CN=A NetBeans Friend, OU=NetBeans, O=netbeans.org, C=US' \n" + | ||
| "keyalg='DSA' \n" + |
There was a problem hiding this comment.
mandatory change to have ant task executed
| "keyalg='DSA' \n" + | ||
| "/>\n" + |
There was a problem hiding this comment.
same keyalgo DSA changes
| <specification-version>6.2</specification-version> | ||
| </run-dependency> | ||
| </dependency> | ||
| <dependency> |
There was a problem hiding this comment.
openide.options is removed so it should be pruned
this indicates that CI isn't running those tests? Should something be added? (tests which don't run don't test anything ;)) |
Would be nice. |
ebarboni
force-pushed
the
bootstrapbuild
branch
from
f68b0e1 to
e11f108
Compare
| if (fail != null && "whenpresent".equals(fail.getValue())) { | ||
| failMsg = "License found in " + f; | ||
| log(failMsg, Project.MSG_ERR); | ||
| } |
There was a problem hiding this comment.
construct failMsg anyway, and log it
|
@mbien the default build-nozip is in fact minimal. What would be the good place in workflow to have such test running? |
2 participants
Hi,
unit test were not working using jdk21 as baseline for bootstrap. changing to release on javac target.
So this is a rewrite of the test using ant buildRule,
Most test pass without changing existing assets,
nbm and jnlp use genkey from ant but needs now algo set. I use DSA not sure it's ok.
fixdependencies uses deprecated openide.options
AFAIK
ant bootrap; ant build-platform;ant localtest are needed to get test compile