-
Notifications
You must be signed in to change notification settings - Fork 49
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
java.lang.NoClassDefFoundError when running tests #133
Comments
Thanks! The current test suite is not up to date. To try out the system I recommend you following the instructions in the readme, grade will skip tests to avoid this issue. Also, the most up-to-date version of this software in the fork on raulcf/aurum-datadiscovery (https://github.com/raulcf/aurum-datadiscovery). |
I was interested in adding another data source. The main reason I was hoping to get tests working was so I could add tests for my own code. I'll try the fork :) |
Makes sense. We're building a new iteration of a data discovery system and
will be open sourcing the code soon, so a lot of resources have shifted,
but happy to help debug, etc.
Raul
…On Tue, Jan 28, 2020 at 9:48 AM Michael Mior ***@***.***> wrote:
I was interested in adding another data source. The main reason I was
hoping to get tests working was so I could add tests for my own code. I'll
try the fork :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#133?email_source=notifications&email_token=ABGQB5LDF6MAVZBQLOIN4ELRABHTHA5CNFSM4KLUDGPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKDZ6CY#issuecomment-579313419>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABGQB5MTCJVKF2ISHX3EBJLRABHTHANCNFSM4KLUDGPA>
.
|
Looks like I'm still getting failing tests in that fork unfortunately. But I'll see if I can make some progress :) However, depending on when the "soon" is, maybe it would be easier for me to wait. |
Currently several tests fail for me.
An example stack trace from one of the failures is below:
The text was updated successfully, but these errors were encountered: