Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

Commit

Permalink
chore(Gruntfile): add grunt-ddescribe-iit task to CI checks
Browse files Browse the repository at this point in the history
This should prevent `ddescribe` and `iit` blocks in tests from being checked into the tree, as they
were in #2311.
  • Loading branch information
caitp authored and Foxandxss committed Jun 9, 2014
1 parent aa8c654 commit 5cd6f4a
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
8 changes: 7 additions & 1 deletion Gruntfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ module.exports = function(grunt) {
grunt.loadNpmTasks('grunt-karma');
grunt.loadNpmTasks('grunt-conventional-changelog');
grunt.loadNpmTasks('grunt-ngdocs');
grunt.loadNpmTasks('grunt-ddescribe-iit');

// Project configuration.
grunt.util.linefeed = '\n';
Expand Down Expand Up @@ -190,12 +191,17 @@ module.exports = function(grunt) {
src: ['src/**/*.js', 'src/**/*.ngdoc'],
title: 'API Documentation'
}
},
'ddescribe-iit': {
files: [
'src/**/*.spec.js'
]
}
});

//register before and after test tasks so we've don't have to change cli
//options on the goole's CI server
grunt.registerTask('before-test', ['enforce', 'jshint', 'html2js']);
grunt.registerTask('before-test', ['enforce', 'ddescribe-iit', 'jshint', 'html2js']);
grunt.registerTask('after-test', ['build', 'copy']);

//Rename our watch task to 'delta', then make actual 'watch'
Expand Down
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@
"grunt-karma": "~0.4.4",
"node-markdown": "0.1.1",
"semver": "~2.2.0",
"shelljs": "~0.2.0"
"shelljs": "~0.2.0",
"grunt-ddescribe-iit": "0.0.4"
},
"license": "MIT"
}

1 comment on commit 5cd6f4a

@pkozlowski-opensource
Copy link
Member

Choose a reason for hiding this comment

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

Thnx @caitp !

Please sign in to comment.