Skip to content

Commit

Permalink
rename build files, add smoke test for build
Browse files Browse the repository at this point in the history
  • Loading branch information
aearly committed Jan 16, 2016
1 parent f984553 commit af0a2ed
Show file tree
Hide file tree
Showing 91 changed files with 44 additions and 1,241 deletions.
4 changes: 3 additions & 1 deletion .jshintrc
Expand Up @@ -25,6 +25,8 @@
"define": true,
"describe": true,
"context": true,
"it": true
"it": true,
"before": true,
"after": true
}
}
9 changes: 6 additions & 3 deletions Makefile
Expand Up @@ -13,14 +13,17 @@ SCRIPTS = ./support
JS_SRC = $(shell find lib/ -type f -name '*.js') package.json
LINT_FILES = lib/ test/ mocha_test/ $(shell find perf/ -maxdepth 2 -type f) support/ gulpfile.js karma.conf.js

UMD_BUNDLE = $(BUILDDIR)/async-bundle.js
CJS_BUNDLE = $(BUILDDIR)/async-cjs.js
UMD_BUNDLE = $(BUILDDIR)/async.js
CJS_BUNDLE = $(BUILDDIR)/index.js

all: lint test clean build

test:
npm test

test-build: build
mocha support/build.test.js

clean:
rm -rf $(BUILDDIR)
rm -rf $(DIST)
Expand All @@ -45,7 +48,7 @@ $(CJS_BUNDLE): $(JS_SRC)

build-dist:
mkdir -p $(DIST)
cp $(BUILDDIR)/async-bundle.js $(DIST)/async.js
cp $(BUILDDIR)/async.js $(DIST)/async.js
$(UGLIFY) $(DIST)/async.js -mc \
--source-map $(DIST)/async.min.map \
-o $(DIST)/async.min.js
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.

2 comments on commit af0a2ed

@megawac
Copy link
Collaborator

Choose a reason for hiding this comment

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

@aearly do we want to have a make file test which copies the test folder, renames the async.js import and tests the built file?

@aearly
Copy link
Collaborator Author

@aearly aearly commented on af0a2ed Feb 16, 2016

Choose a reason for hiding this comment

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

I added support/build.test.js in a later commit that does a simple smoke test of the build artifacts.

Please sign in to comment.