This is an automated email from the ASF dual-hosted git repository.
zjffdu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zeppelin.git
The following commit(s) were added to refs/heads/master by this push:
new a183ec3 [ZEPPELIN-3898] Adding repo in testDelRepo so test can run by
itself
a183ec3 is described below
commit a183ec3bd4b6fd1da2a3b695000fdf1a391f2458
Author: OrDTesters <[email protected]>
AuthorDate: Tue Dec 4 00:46:01 2018 -0600
[ZEPPELIN-3898] Adding repo in testDelRepo so test can run by itself
### What is this PR for?
Test DependencyResolverTest.testDelRepo fails when run on its own. The test
needs a repo to already be added so it can delete it. If the test is run on its
own, the repo is not added to start and so the test fails. The proposed PR
fixes this issue by having the test add the repo at the beginning.
### What type of PR is it?
Bug Fix
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3898
Author: OrDTesters <[email protected]>
Closes #3249 from OrDTesters/fix-DependencyResolverTest-testDelRepo and
squashes the following commits:
13a282252 [OrDTesters] [ZEPPELIN-3898] Adding repo in testDelRepo so test
can run by itself
---
.../src/test/java/org/apache/zeppelin/dep/DependencyResolverTest.java | 1 +
1 file changed, 1 insertion(+)
diff --git
a/zeppelin-interpreter/src/test/java/org/apache/zeppelin/dep/DependencyResolverTest.java
b/zeppelin-interpreter/src/test/java/org/apache/zeppelin/dep/DependencyResolverTest.java
index 7ccc7df..1dbe94b 100644
---
a/zeppelin-interpreter/src/test/java/org/apache/zeppelin/dep/DependencyResolverTest.java
+++
b/zeppelin-interpreter/src/test/java/org/apache/zeppelin/dep/DependencyResolverTest.java
@@ -66,6 +66,7 @@ public class DependencyResolverTest {
@Test
public void testDelRepo() {
+ resolver.addRepo("securecentral", "https://repo1.maven.org/maven2";, false);
int reposCnt = resolver.getRepos().size();
resolver.delRepo("securecentral");
resolver.delRepo("badId");
