Skip to content
This repository was archived by the owner on Jul 10, 2024. It is now read-only.

SUBMARINE-69. Add tests to ZipUtilities class#112

Closed
adamantal wants to merge 1 commit intoapache:masterfrom
adamantal:SUBMARINE-69
Closed

SUBMARINE-69. Add tests to ZipUtilities class#112
adamantal wants to merge 1 commit intoapache:masterfrom
adamantal:SUBMARINE-69

Conversation

@adamantal
Copy link
Copy Markdown

What is this PR for?

Added unit tests to ZipUtilities

What type of PR is it?

Test

Todos

  • - new UTs should pass

What is the Jira issue?

SUBMARINE-69

How should this be tested?

  • New unit tests should pass.
  • No further testing is needed.

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

Copy link
Copy Markdown
Member

@xunliu xunliu left a comment

Choose a reason for hiding this comment

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

LGTM

@asfgit asfgit closed this in f82ca81 Nov 29, 2019
kevin85421 pushed a commit to kevin85421/hadoop-submarine that referenced this pull request Dec 24, 2019
### What is this PR for?
Added unit tests to `ZipUtilities`

### What type of PR is it?
Test

### Todos
* [ ] - new UTs should pass

### What is the Jira issue?
[SUBMARINE-69](https://issues.apache.org/jira/browse/SUBMARINE-69)

### How should this be tested?
* New unit tests should pass.
* No further testing is needed.

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No

Author: Adam Antal <adam.antal@cloudera.com>

Closes apache#112 from adamantal/SUBMARINE-69 and squashes the following commits:

9f66792 [Adam Antal] SUBMARINE-69. Add tests to ZipUtilities class
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants