-
Notifications
You must be signed in to change notification settings - Fork 139
Setup OSX CI & use externalproject for GMock and GTest & cucmber 1.3.20 #86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
2e09eb6
5b0ea92
3bb7d32
fab7724
e4414cf
0e76db6
bc2cd11
bdcc9e2
5a409f7
663fa4b
fa77eac
2a6cd53
fc52985
4a8a81d
d72b7ba
17933a0
dba01a5
ff1ad75
b896715
de51ee9
aa6aa1e
72d6dcc
1226cf3
c847396
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,11 +1,27 @@ | ||
| language: cpp | ||
|
|
||
| sudo: required | ||
| dist: trusty | ||
| os: | ||
| - linux | ||
| - osx | ||
| compiler: | ||
| - clang | ||
| - gcc | ||
| rvm: | ||
| - 1.9.3 | ||
| - 2.0 | ||
| - 2.2 | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. it does not seem to work (always runs with 2.2.3), probably because it's a c++ project
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. After all Ruby deps are there just to run our tests. We shouldn't be concerned with testing multiple Ruby versions. Let's just choose one thst works with rvm (.ruby-version).
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes - I noticed that it doesn't work. And agree that it is probably because it's not ruby project for travis. I left it there to think about it later, but then forgot about that... |
||
| addons: | ||
| apt: | ||
| # List of whitelisted in travis packages for ubuntu-precise can be found here: | ||
| # https://github.com/travis-ci/apt-package-whitelist/blob/master/ubuntu-precise | ||
| # List of whitelisted in travis apt-sources: | ||
| # https://github.com/travis-ci/apt-source-whitelist/blob/master/ubuntu.json | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. don't think comments are needed here: the whitelisting process is quite clear in the Travis documentation
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Of course. TBH it was copy-paste from google-test .travis.yml |
||
| packages: | ||
| - libboost-thread-dev | ||
| - libboost-system-dev | ||
| - libboost-regex-dev | ||
| - libboost-date-time-dev | ||
| - libboost-test-dev | ||
|
|
||
| before_script: | ||
| - "sudo apt-get install libboost-thread-dev libboost-system-dev libboost-regex-dev libboost-date-time-dev libboost-test-dev google-mock" | ||
| - "cd /usr/src/gtest && sudo cmake . && sudo cmake --build . && sudo mv libg* /usr/local/lib/ ; cd -" | ||
|
|
||
| script: "cmake -E make_directory build && cmake -E chdir build cmake -DCUKE_ENABLE_EXAMPLES=on .. && cmake --build build && cmake --build build --target test && cmake --build build --target features" | ||
| script: ./travis.sh | ||
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -35,7 +35,7 @@ class BeforeHook : public Hook { | |
|
|
||
| class AroundStepHook : public Hook { | ||
| public: | ||
| virtual void invokeHook(Scenario *scenario, CallableStep *step); | ||
| virtual void invokeHookWithStep(Scenario *scenario, CallableStep *step); | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. why this name change? there is only one invokeHook and I don't see the point of renaming it
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It is from pull request #72 |
||
| virtual void skipHook(); | ||
| protected: | ||
| CallableStep *step; | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -62,6 +62,11 @@ class InvokeArgs { | |
|
|
||
| template<class T> T getInvokeArg(size_type i) const; | ||
| const Table & getTableArg() const; | ||
|
|
||
| size_type argsSize() const { | ||
| return args.size(); | ||
| } | ||
|
|
||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is this used anywhere?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It is from pull request #72
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hi, sorry to chime in here. I've used this extension in other code, not pushed or published anywhere (yet). I needed it to pass it on to another code layer (if I recall correctly).
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd say it belongs to a PR of its own (but it's hard to know if it makes sense without knowing what it should be used for) or it should live in your own branch. It didn't make any sense fixing the compiler warnings or in the OSX build.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sure. I guess I've to rebase my stuff on master once all this and other fixes landed. I've to review my code (which laid dormant for some time now), and will check what this function was required for. In the context of the other changes this should be dropped therefore (I guess)
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Apart from the Boost improvements that should have come from another PR, I think this is the only change that I didn't apply in #107. Once that is merged, it's probably easier to apply this single change on top of master instead of rebasing (that took me quite some time). |
||
| private: | ||
| Table tableArg; | ||
| args_type args; | ||
|
|
@@ -91,7 +96,7 @@ class InvokeResult { | |
|
|
||
| bool isSuccess() const; | ||
| bool isPending() const; | ||
| const InvokeResultType getType() const; | ||
| InvokeResultType getType() const; | ||
| const std::string &getDescription() const; | ||
| }; | ||
|
|
||
|
|
@@ -157,7 +162,7 @@ class StepManager { | |
| }; | ||
|
|
||
|
|
||
| static std::string toSourceString(const char *filePath, const int line) { | ||
| static inline std::string toSourceString(const char *filePath, const int line) { | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. what kind of performance improvement do you expect by inlining this? or was it done for a different reason?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It is from pull request #72 |
||
| using namespace std; | ||
| stringstream s; | ||
| string file(filePath); | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| #rvm install 2.2 | ||
| ruby -v | ||
| gem install bundler | ||
| rm Gemfile.lock | ||
| rm build -rf | ||
| cmake -E make_directory build | ||
| cmake -E chdir build cmake -DCUKE_ENABLE_EXAMPLES=on -DCUKE_DISABLE_BOOST_TEST=on .. | ||
| cmake --build build | ||
| cmake --build build --target test | ||
| cmake --build build --target features | ||
| build/examples/Calc/GTestCalculatorSteps >/dev/null & | ||
| cucumber examples/Calc | ||
| killall GTestCalculatorSteps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's create a pull request only for OSX (it's not working)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK