Skip to content

Commit

Permalink
Fix make start and UT
Browse files Browse the repository at this point in the history
  • Loading branch information
mohsin-plivo committed Oct 17, 2024
1 parent dfa06d1 commit 4e8ed57
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
6 changes: 5 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,11 @@ build:

start:
docker-compose up --build --remove-orphans --detach
docker attach $(shell docker-compose ps -q javaSDK)
# Wait for the container to be running before attaching
@while [ -z "$$(docker-compose ps -q javaSDK)" ]; do \
sleep 1; \
done
docker attach $$(docker-compose ps -q javaSDK)

test:
@[ "${CONTAINER}" ] && \
Expand Down
4 changes: 2 additions & 2 deletions src/test/java/com/plivo/api/VerifySessionTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -28,15 +28,15 @@ public void setUp() throws Exception {
public void invalidSessionCreated() throws Exception {
expectResponse("createSession.json", 202);

VerifySession.creator(null,null, null, null, null,null, null, null, 0, null)
VerifySession.creator(null,null, null, null, null,null, null, null, 0, null, null)
.create();
}

@Test
public void sessionCreated() throws Exception {
expectResponse("createSession.json", 202);

VerifySession.creator(null,"+1234567890", null, null, null, null, null, null, 0, null)
VerifySession.creator(null,"+1234567890", null, null, null, null, null, null, 0, null, null)
.create();
}

Expand Down

0 comments on commit 4e8ed57

Please sign in to comment.