Skip to content
This repository has been archived by the owner. It is now read-only.

fix BasicWorkflow DocSig assertion #1707

Merged
merged 1 commit into from Jan 21, 2017

Conversation

@sugakandrey
Copy link
Contributor

commented Jan 21, 2017

No description provided.

@@ -121,7 +121,10 @@ class BasicWorkflow extends EnsimeSpec
expectMsg(Some(intDocSig))

project ! intDocSig
expectMsgType[StringResponse].text should endWith("/index.html#scala.Int")
expectMsgType[StringResponse].text should (
endWith("/index.html#scala.Int") or // <= 2.11

This comment has been minimized.

Copy link
@fommil

fommil Jan 21, 2017

Contributor

this or stuff is awesome. I never knew about it until you started using it

@fommil
fommil approved these changes Jan 21, 2017
@fommil

This comment has been minimized.

Copy link
Contributor

commented Jan 21, 2017

timeouts in the debugtest, transient. Probably just need to bump the timeouts or scaling factor.

@fommil fommil merged commit 63cf102 into ensime:2.0 Jan 21, 2017

2 of 4 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/drone/pr this build is pending
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
licence/cla Contributor License Agreement is signed.
Details
sugakandrey added a commit to sugakandrey/ensime-server that referenced this pull request Jan 25, 2017
Merge branch '2.0' into 2.0-graph
* 2.0:
  remove superflous test
  scala 2.12.1 support
  fix BasicWorkflow DocSig assertion (ensime#1707)
  2.12.0 support (not 2.12.1... yet) (ensime#1703)
  aggressive timeouts in tests (ensime#1704)
  Add EnsimeProject for .ensime v2 format (ensime#1693)
  no need for mkdir /root/.ssh in drone build
  bump headers (ensime#1701)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants
You can’t perform that action at this time.