Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

Scala system tests #206

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

Scala system tests #206

wants to merge 7 commits into from

Conversation

peller
Copy link
Collaborator

@peller peller commented Feb 3, 2016

No description provided.

@peller peller changed the title Scala system test [WIP] Scala system test Feb 4, 2016
@peller peller force-pushed the ScalaSystemTest branch 3 times, most recently from 416e645 to d73c5d6 Compare February 4, 2016 16:21
@lbustelo lbustelo force-pushed the ScalaSystemTest branch 2 times, most recently from 5236547 to 4a2b51d Compare April 1, 2016 20:09
@poplav poplav force-pushed the ScalaSystemTest branch 14 times, most recently from 403d9f2 to ff70ae5 Compare April 12, 2016 15:59
@poplav
Copy link
Contributor

poplav commented Apr 12, 2016

One of the intermittent errors is the result of a compilation/interpreter error when invoking a function. As a setup to the tests we run all cells and ensure all finished executing in order to proceed to the actual tests of correct output. In this case the kernel will fail on cell five urth-core-bind where we invoke a simple function and fail to execute any other cells/reach the tests. The source is unknown whether widgets or toree.
screen shot 2016-04-12 at 3 55 04 pm

@lbustelo
Copy link
Collaborator

@poplav And the error above cannot be replicated locally?

@poplav
Copy link
Contributor

poplav commented Apr 21, 2016

This error appears locally a very small percent of the time, but more frequently on travis. I think this error is hidden when run on travis, but explains why it will get stuck on a run all in travis (where we are logging the number of output areas/cells executed). This is a compiler/interpreter error when the function is invoked, which is very strange/not sure if this deals with toree.
There also is an intermittent issue I believe with the channel init message reaching the kernel on initial page load and init of widgets. If it is not inited by the time we invoke channel this.channel will be null as it hasn't been registered. This issue goes away though when the page is refreshed and the init channel message finally reaches the kernel. Making channel set lazy solved for the message arriving after a run all, but in this case it wouldn't arrive until you do a page refresh. Would the PR regarding require user to init change this behavior once it gets merged in?

@poplav poplav force-pushed the ScalaSystemTest branch 3 times, most recently from 16a4dcc to 971ca7f Compare April 28, 2016 20:59
@lbustelo
Copy link
Collaborator

@poplav now that the init code is part of master... can you try and rebase this branch and see if it start running reliably?

wellecks and others added 6 commits May 23, 2016 13:06
(c) Copyright IBM Corp. 2016
…eding with tests

use localhost:8888 for addjar

(c) Copyright IBM Corp. 2016
(c) Copyright IBM Corp. 2016
@poplav poplav force-pushed the ScalaSystemTest branch 5 times, most recently from 2e70325 to 62371fe Compare May 24, 2016 16:17
@poplav poplav closed this May 24, 2016
@poplav poplav reopened this May 24, 2016
@poplav poplav force-pushed the ScalaSystemTest branch 11 times, most recently from 3c13509 to aa048a3 Compare May 25, 2016 20:43
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.

None yet

4 participants