Skip to content

Conversation

@jamesagardner
Copy link
Contributor

No description provided.

cskeefer and others added 9 commits July 22, 2019 14:51
Fixing issue where empty sequences were not being considered equal to one another when using assert-equal-json().
I am running "gradle -i bintrayUpload" from the root project directory, and all 8 artifacts are published seemingly successfully, but the bintray plugin still throws an error. The notes on the 1.8.4 release imply that 1.8.4 fixes a bug regarding publishing something multiple times, but that appears to be the error. But, the packages still seem to be uploaded successfully.
- Description of marklogic-unit-test and marklogic-junit
- Links to sample projects
- Moving contributing data to wiki
This allows for marklogic-junit5 to be published under marklogic-community
Adding instructions for getting started using marklogic-unit-test.
- Test names that started with the suite name no longer throw a module
  not found error.
- Adding a better failure message
Changing Travis CI distribution to Trusty which still supports Oracle JDK 8
@jamesagardner
Copy link
Contributor Author

I think the above failure is pointing out that the master branch still has the old Travis CI distribution, which is no longer working with Oracle JDK 8. I believe the commit to use the trusty distribution fixes the issue (travis-ci/pr passed, but travis-ci/pull failed).

So I think we're OK to continue with the merge to master assuming the code review passes. I'm hoping to finish this PR on Monday and release 1.0.0 at that time.

Copy link
Contributor

@hansenmc hansenmc left a comment

Choose a reason for hiding this comment

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

I found a handful of other things that should probably be adjusted.

I submitted them as separate PRs, so you can decide whether to pull in now or if you want to defer and pull them in post 1.0 release.

Correcting the JavaDocs. Several methods had @param that did not exist
Questions or Problem linking to non-existent anchor. Removing it.
…orms (#95)

Can now take advantage of the test-dir parameter in an XSLT transform.
name() will return the namespace-prefix of the instance XML document, which may not use "error". Changing to use the self:: axis and use XPath to test if it is an error:error element.
No need to repeat the type on the right hand side
@hansenmc
Copy link
Contributor

hansenmc commented Sep 8, 2019

I noticed that the marklogic-unit-test-modules test-helper.xqy references "ROXY-BOOM":

https://github.com/marklogic-community/marklogic-unit-test/blob/develop/marklogic-unit-test-modules/src/main/ml-modules/root/test/test-helper.xqy#L50

Not sure if we want to leave that in there as an ode to it's origins, or if we wanted to remove all references to Roxy.

@hansenmc
Copy link
Contributor

hansenmc commented Sep 8, 2019

Also, before publishing the copyright year should be updated to the current year. There are several files that have copyright (c) 2018 MarkLogic Corporation

hansenmc and others added 8 commits September 13, 2019 09:42
…acter encoding of the system to convert the string to a byte array. (#98)

Instead, use new InputSource(new StringReader(xml))

The StringReader avoids having to convert to a byte array and will send the character stream to the InputSource, which will determine the encoding through standard XML API parse rules.
* Remove unnecessary declaration of logger that is shadowing the logger from LoggingObject that it extends.

* Remove unused imports (now that logger is not explicitly initialized here)
@jamesagardner jamesagardner merged commit 63a8aa4 into master Sep 13, 2019
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.

6 participants