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

Feature/kartothek table name tests #465

Open
wants to merge 82 commits into
base: master
Choose a base branch
from

Conversation

MartinHaffner
Copy link

Added some tests to check the behaviour:

  • of reading or writing/Updating
  • cubes or datasets
  • with or without the explicit usage of a table name, and with the usage of a non-standard table name

In some cases the interface to specify an alternative table name does not yet or does no more exist; these tests were added for the sake of completeness, but will be skipped.
In some other cases, KTK failed with an unspecific error where a graceful failure with a meaningful Error was expected; these are marked as xfails.

This PR depends on

BY internal: Refers to JIRA Task LDE-1084.

Doc build fails in the forked repo at the moment; but I was told that in some cases it would work in the upstream repo anyway, so I'm just giving it a try...

martin-haffner-by and others added 30 commits April 15, 2021 09:40
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from v1.3.1 to v1.4.0.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/master/CHANGELOG.md)
- [Commits](codecov/codecov-action@v1.3.1...0e28ff8)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants