Skip to content
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

implement add file from buffer #196

Merged
merged 3 commits into from Oct 24, 2021

Conversation

AlinaKul
Copy link
Contributor

No description provided.

await wb.xlsx.load(buffer);

return new Importer(wb)

Choose a reason for hiding this comment

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

whitespace

@@ -26,7 +27,19 @@ describe('reading sigle items (objects)', () => {
chai.expect(result).eql(expected);
chai.expect(result.length).equals(1);
});
it('getAllItems should return one correct object from buffer', async () => {
const factory = new ImporterFactory();
const fs = require('fs')

Choose a reason for hiding this comment

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

Import should be moved to the top and use import instead of require to be consistent with other imports.

@Siemienik Siemienik self-requested a review October 20, 2021 05:32
@codecov
Copy link

codecov bot commented Oct 20, 2021

Codecov Report

Merging #196 (26007a8) into master (c7bac21) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #196   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           55        55           
  Lines         1063      1078   +15     
  Branches       154       154           
=========================================
+ Hits          1063      1078   +15     
Flag Coverage Δ
xlsx-import 100.00% <100.00%> (ø)
xlsx-renderer 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/xlsx-import/src/ImporterFactory.ts 100.00% <100.00%> (ø)
packages/xlsx-import/src/mappers/dateMapper.ts 100.00% <100.00%> (ø)
...port/tests/integration/object-single-items.test.ts 100.00% <100.00%> (ø)

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 c7bac21...26007a8. Read the comment docs.

@kibertoad
Copy link

@Siemienik Is there anything missing for this PR?

@Siemienik
Copy link
Owner

@AlinaKul @kibertoad Thank you for the contribution, that's awesome 😄 . I need time to test it and I'm going to do it at the weekend.

@sonarcloud
Copy link

sonarcloud bot commented Oct 24, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Siemienik Siemienik merged commit d68a772 into Siemienik:master Oct 24, 2021
@Siemienik
Copy link
Owner

Siemienik commented Oct 24, 2021

@AlinaKul, great contribution 🥇 Thank you.
In days, I publish to npm new importer version (first, I've to update the documentation). I will notify about it on Gitter: https://gitter.im/Siemienik/community

@AlinaKul
Copy link
Contributor Author

@Siemienik happy to help you!
hope this contribution will be helpful for all users.
Waiting very much for a new version.
And of course BIG Thanks @kibertoad for helping me !!!

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.

None yet

3 participants