Skip to content

Version 0.1.29#23

Merged
inbaltako merged 7 commits intomasterfrom
dev
Oct 1, 2020
Merged

Version 0.1.29#23
inbaltako merged 7 commits intomasterfrom
dev

Conversation

@inbaltako
Copy link
Copy Markdown
Contributor

No description provided.

@inbaltako inbaltako requested a review from alexivsn September 30, 2020 10:22
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 30, 2020

Codecov Report

Merging #23 into master will increase coverage by 7.66%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master       #23      +/-   ##
===========================================
+ Coverage   92.33%   100.00%   +7.66%     
===========================================
  Files          46         1      -45     
  Lines         913        13     -900     
===========================================
- Hits          843        13     -830     
+ Misses         70         0      -70     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2d4cb35...af303df. Read the comment docs.

Comment thread lib/securenative/sdk.rb Outdated
@api_manager = ApiManager.new(@event_manager, @options)
SecureNativeLogger.init_logger(@options.log_level)
end
class SecureNativeSDK
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing module ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since it's unique (no other class is under the name SecureNativeSDK using a module will be redundant --> SecureNative::SecureNativeSDK

Comment thread securenative.gemspec Outdated
Comment thread securenative.gemspec
spec.version = '0.1.29'
spec.authors = ['SecureNative']
spec.email = ['support@securenative.com']

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to define spec. test_files = Dir['spec//*']
And regarding spec.files, do u really need this Dir.chdir(File.expand_path('..', FILE)) ?
why not something like this spec.files = Dir['{lib}/
/*'] + ['README.md']

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added spec. test_files, spec folder also contains a spec help file which defines tests.

This file was autogenerated by IntelliJ, I guess it's they're way of specifying files (using git), anyway i don't know if using spec.files = Dir['{lib}//*'] + ['README.md'] will do it, still missing important files like Gemfile and securenative.spec not sure it will work bundles like that

Comment thread securenative.gemspec
Comment thread spec/spec_api_manager.rb
Comment thread lib/securenative/event_manager.rb
Comment thread lib/securenative/event_manager.rb Outdated
Comment thread lib/securenative/event_manager.rb Outdated
Comment thread lib/securenative/event_manager.rb Outdated
Comment thread lib/securenative/event_manager.rb
@inbaltako inbaltako merged commit 682e43f into master Oct 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants