Merge pull request #4 from WittBulter/chore-build

 build(pkg): v0.1.2

Author: WittBulter

.travis.yml

@@ -1,6 +1,10 @@
 
 language: node_js
 
+cache:
+  directories:
+    - "node_modules"
+
 notifications:
   email:
     on_success: never
@@ -15,7 +19,6 @@ branches:
 before_install: git fetch --depth=1000
 
 install:
-  - npm i -g typescript webpack
   - npm i
 
 script:

README.md

@@ -4,7 +4,7 @@
 </div>
 
 <br><br><br>
-### back-loader
+### back-loader (~ 1.2kb gzipped)
 > a easy and steady preload lib, you can use it to load all kinds of resources. 
 
 >use **back-loader** in your project, you can quickly preload more pictures, styles(.css) or scripts, it will make your website more fluent.

build/main/webpack.prod.ts

@@ -13,6 +13,8 @@ module.exports = {
     filename: '[name].js',
   },
 
+  target: 'web',
+  
   resolve: {
     extensions: ['.ts', '.js'],
   },
@@ -38,16 +40,13 @@ module.exports = {
         ],
       },
     ],
-    
-    exprContextCritical: false,
   },
 
   plugins: [
     new webpack.NoEmitOnErrorsPlugin(),
 
     new webpack.optimize.UglifyJsPlugin({
-      mangle: { keep_fnames: true },
-      sourceMap: false,
+      compress: { warnings: false },
     }),
   ],
 }

examples/main.ts

@@ -5,7 +5,7 @@ const handler = async() => {
    const backLoader = new BackLoader({
      // scripts: ['https://code.jquery.com/jquery-3.2.1.slim.min.js'],
      // pages: ['https://google.com'],
-     images: ['https://cn.bing.com/az/hprichbg/rb/BarHarborCave_ZH-CN8055769470_1920x1080.jpg'],
+     images: ['https://www.bing.com/az/hprichbg/rb/TadamiTrain_ROW14602114613_1920x1080.jpg'],
    })
   backLoader.start().on(event => {
     console.log(event)

package-lock.json

@@ -1,19 +1,19 @@
 {
   "name": "back-loader",
-  "version": "0.1.0",
-  "description": "",
-  "main": "release/index.d.ts",
-  "unpkg": "release/back-loader.js",
+  "version": "0.1.3",
+  "description": "a easy and steady preload lib",
+  "main": "release/index",
+  "unpkg": "release/index.js",
   "author": "WittBulter(nanazuimeng123@gmail.com)",
   "homepage": "https://github.com/DhyanaChina/back-loader#readme",
   "license": "MIT",
   "scripts": {
     "test": "./node_modules/.bin/mocha",
-    "cover:local": "./node_modules/.bin/nyc mocha && open coverage/lcov-report/index.html",
-    "cover:hook": "./node_modules/.bin/nyc mocha && cat ./coverage/lcov.info | coveralls",
+    "cover:local": "./node_modules/.bin/nyc mocha;open coverage/lcov-report/index.html",
+    "cover:hook": "./node_modules/.bin/nyc mocha;cat ./coverage/lcov.info | coveralls",
     "ex": "tsc ./build/webpack.dev.ts && env DEBUG=true webpack-dev-server --config build/webpack.dev.js --inline --progress --port 1338 && npm run clean:tsc",
     "ex:build": "tsc ./build/webpack.prod.ts && rm -rf dist && webpack --config build/webpack.prod.js && npm run clean:tsc",
-    "release": "tsc ./build/main/webpack.prod.ts && rm -rf release && webpack --config build/main/webpack.prod.js && npm run clean:tsc",
+    "release": "tsc -p ./tsconfig.json",
     "clean:tsc": "find build -name '*.js' | xargs rm -f",
     "clean:cover": "rm -rf coverage .nyc_output",
     "clean:build": "rm -rf dist release",
@@ -42,6 +42,7 @@
     "@types/html-webpack-plugin": "^2.30.1",
     "@types/jsdom": "^11.0.4",
     "@types/mocha": "^2.2.46",
+    "@types/parse5": "^3.0.0",
     "@types/webpack": "^3.8.2",
     "@types/webpack-merge": "^4.1.1",
     "awesome-typescript-loader": "^3.4.1",

package.json

@@ -39,22 +39,28 @@ export class EventHub {
     // just remove one
     if (done) return this.removeOne(eventType, done)
     // remove this type
-    this.eventRecords
-      .filter(re => re.type === eventType)
-      .forEach(re => this.source.removeEventListener(re.type, re.handle))
+    this.eventRecords = this.eventRecords
+    .map(re => {
+      if (re.type !== eventType) return re
+      this.source.removeEventListener(re.type, re.handle)
+      return null
+    })
+    .filter(r => r)
   }
 
   removeAll(): void {
     this.eventRecords
-      .forEach(re => this.source.removeEventListener(re.type, re.handle))
+    .forEach(re => this.source.removeEventListener(re.type, re.handle))
+    this.eventRecords = []
   }
 
   private removeOne(eventType: string, done?: EventHubListener): void {
-    const record: EventRecord = this.eventRecords.find(record => {
+    const index: number = this.eventRecords.findIndex(record => {
       return record.type === eventType && record.done === done
     })
-    if (!record) return
-    this.source.removeEventListener(eventType, record.handle)
+    if (index < 0) return
+    this.source.removeEventListener(eventType, this.eventRecords[index].handle)
+    this.eventRecords.splice(index, 1)
   }
 
 }

src/core/event.ts

@@ -22,12 +22,12 @@ export class Loader {
 
   pages(urls: string[]): void {
     urls.forEach(url => {
-      $fetch(url, { mode: 'cors' })
+      $fetch(url, { mode: 'no-cors' })
       .then(html => {
         this.scripts(filterResources(html, 'script'))
         this.styles(filterResources(html, 'style'))
       })
-      .catch(() => {
+      .catch((e) => {
         this.emit(Object.assign({}, this.baseEvent, {
           type: 'page', source: url, success: false,
         }))

src/core/loader.ts

@@ -1,7 +1,7 @@
 // resource load use no-cors mode default
 // html load must use cors
 export const $fetch = (url: string, init: RequestInit = {}) => {
-  return fetch(url, Object.assign({ mode: 'no-cors' }, init))
+  return fetch(url, Object.assign({ mode: 'cors', cache: 'reload' }, init))
   .then(r => r.text())
 }
 

src/utils/tools.ts

@@ -23,4 +23,19 @@ describe('Core function test', () => {
     hub.dispath('test_listen_2', 'once')
   })
 
-})
+  it('should nerve receive any notice', done => {
+    hub.listen('test_listen_3', (e, d) => expect.fail(d, null))
+    hub.removeAll()
+    hub.dispath('test_listen_3', 'once')
+    setTimeout(done, 100)
+  })
+  
+  it('get an event and reject an event', done => {
+    hub.listen('test_listen_4', () => process.exit(1))
+    hub.listen('test_listen_5', () => done())
+    hub.remove('test_listen_4')
+    hub.dispath('test_listen_4', 'once')
+    hub.dispath('test_listen_5', 'once')
+  })
+  
+})