os-fonts - npm Package Compare versions

Comparing version 0.3.0 to 0.3.1

__tests__/index.js

 import m from '../';
 
+jest.mock('fs');
+
 describe('os-fonts', () => {
-  it('should get all fonts', async () => {
+  const fs = require('fs');
+
+  it('should retrieve some fonts', async () => {
+    // set up mock data
+    fs.__setFiles([
+      'foo.ttf',
+      'bar.ttf',
+    ]);
     const fonts = await m.getAll();
-    expect(fonts.length).toBeTruthy();
+    expect(fonts.length).toBe(2);
   });
 
-  it('should get all user fonts', async () => {
+  it('should retrieve no fonts', async () => {
+    fs.__setFiles([]);
+    const fonts = await m.getAll();
+    expect(fonts.length).toBe(0);
+  });
+
+  it('should retrieve user fonts', async () => {
+    // set up mock data
+    fs.__setFiles([]);
     const fonts = await m.getAll('user');
-    expect(fonts.length).toBeTruthy();
+    expect(fonts.length).toBe(0);
   });
+
+  it('should get all network fonts', async () => {
+    // set up mock data
+    fs.__setFiles([
+      'foo.ttf',
+      'bar.ttf',
+      'baz.ttf',
+      'qux.ttf',
+    ]);
+    const fonts = await m.getAll('network');
+    expect(fonts.length).toBe(4);
+  });
+
+  it('should return nothing for readdir errors', async () => {
+    fs.__setReturnError(true);
+    const fonts = await m.getAll('network');
+    expect(fonts.length).toBe(0);
+  });
 });

index.js

@@ -38,6 +38,7 @@ const path = require('path');
  */
-exports.getFontsInDirectory = dir => new Promise((resolve, reject) => {
+exports.getFontsInDirectory = dir => new Promise((resolve) => {
   fs.readdir(dir, (err, files) => {
     if (err) {
-      reject(err);
+      resolve([]);
+      return;
     }
@@ -44,0 +45,0 @@ resolve(files.map(f => path.join(dir, f)));

package.json

 {
   "name": "os-fonts",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Retrieve fonts available on your OS.",
@@ -5,0 +5,0 @@ "keywords": [

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

111255

increased by1.32%

Number of package files

13

increased by8.33%

Lines of code

122

increased by103.33%

Worsened metrics

Number of low supply chain risk alerts

7

increased by16.67%

No dependency changes