Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

RESTful API tests #92

Closed
s-oravec opened this issue Jun 7, 2014 · 4 comments
Closed

RESTful API tests #92

s-oravec opened this issue Jun 7, 2014 · 4 comments

Comments

@s-oravec
Copy link

s-oravec commented Jun 7, 2014

I would like to see out of box support for RESTful API tests.
Currently it isn't possible without some initial hacking as express is initialized asynchronously and session store is not ready yet when the tests start (or maybe it is my lack of js coding skills ;)) or by running mocha tests in separate grunt task.
@amoshaviv
Copy link
Contributor

Hi @s-oravec,

Did you check SuperTest: https://github.com/visionmedia/supertest it's an assertion library built specially for HTTP assertions. Can be easily integrated with mocha.

@ilanbiala
Copy link
Member

Supertest has been useful for me in testing routes before, I can help add that feature in if you guys would like.

rschwabco added a commit that referenced this issue Oct 21, 2014
@rschwabco
Merge pull request #189 from w-green/adding_server_route_tests
a799a23 
Feature: Article CRUD ops. Added supertests. closes #92
@FREEZX
Copy link

FREEZX commented Jan 5, 2015

Can we get a generator for this in https://github.com/meanjs/generator-meanjs/ ?

@FREEZX
Copy link

FREEZX commented Jan 5, 2015

Added in my fork: https://github.com/FREEZX/generator-meanjs/blob/master/express-test/templates/_.server.routes.test.js

pdfowler added a commit to pdfowler/mean that referenced this issue Jan 19, 2016
@pdfowler
Merge branch 'master' of https://github.com/meanjs/mean into develop …
0dcd0e1 
…to update to MEAN.JS v 0.3.3

# By Liran Tal (10) and others
# Via Roie Cohen (8) and others
* 'master' of https://github.com/meanjs/mean: (24 commits)
  removing un-necessary console.log() calls
  adding chalk to package.json, locking to 0.5 version as a known and compliant API
  adding chalk for formatting console messages instead of ANSI escape codes
  removing un-necessary .js extension
  fixing up broken links with exchange for newer documentation links on expressjs.com
  adding ios rendering support for mobile browsers per bug report meanjs#194
  replacing deprecated require.extensions nodejs call with explicit full paths for require()
  removing the comment too, it may be confusing to think it should just be enabled for JSONP to be globally enabled which isnt true
  disabling JSONP from controllers and commenting out from expressjs configuration, allowing users to enable if they need to
  correcting spellings and English for main README file
  Bump minor version
  Small Update
  Tiny fixes
  Feature: Article CRUD ops. Added supertests. closes meanjs#92
  Use ng-annotate instead of ngmin
  Removed localhost from env files, as it is not necesary and can introduce bugs in non-local environments.
  moving security configs to secure code block to remove requirement for private key and certificate
  Fixed typo of div attribute
  Rise gunt-concurrent task limit.
  adding secure.js environment option, reverting production.js, and adding openssl options to gen-certs
  ...

Conflicts:
	.gitignore
	app/controllers/articles.server.controller.js
	app/controllers/errors.server.controller.js
	app/controllers/users.server.controller.js
	app/controllers/users/users.authentication.server.controller.js
	app/controllers/users/users.password.server.controller.js
	app/controllers/users/users.profile.server.controller.js
	app/routes/articles.server.routes.js
	app/routes/core.server.routes.js
	app/views/layout.server.view.html
	config/env/all.js
	config/env/development.js
	config/env/production.js
	config/express.js
	config/strategies/local.js
	gruntfile.js
	package.json
	public/dist/application.js
	public/modules/users/views/authentication/signin.client.view.html
kgrossnickle pushed a commit to cen3031-5a/trainerScheduling that referenced this issue Oct 10, 2017
Feature: Article CRUD ops. Added supertests. closes meanjs/mean#92
5542d96
kgrossnickle pushed a commit to cen3031-5a/trainerScheduling that referenced this issue Oct 10, 2017
Merge pull request #189 from w-green/adding_server_route_tests
cd27f37 
Feature: Article CRUD ops. Added supertests. closes meanjs/mean#92
jspyhotdev pushed a commit to jspyhotdev/MEAN-board that referenced this issue Aug 14, 2018
@warreng
Feature: Article CRUD ops. Added supertests. closes meanjs/mean#92
21cf061
jspyhotdev pushed a commit to jspyhotdev/MEAN-board that referenced this issue Aug 14, 2018
@rschwabco
Merge pull request #189 from w-green/adding_server_route_tests
eb70f7a 
Feature: Article CRUD ops. Added supertests. closes meanjs/mean#92
@snyk-bot snyk-bot mentioned this issue May 29, 2021
Open
@snyk-bot snyk-bot mentioned this issue Jun 18, 2021
Open
lupinthethirdgentleman pushed a commit to lupinthethirdgentleman/mean-dashboard that referenced this issue Aug 5, 2021
Feature: Article CRUD ops. Added supertests. closes meanjs/mean#92
6ed6cb8
lupinthethirdgentleman pushed a commit to lupinthethirdgentleman/mean-dashboard that referenced this issue Aug 5, 2021
Merge pull request #189 from w-green/adding_server_route_tests
b1e4f35 
Feature: Article CRUD ops. Added supertests. closes meanjs/mean#92
@tejas002 tejas002 mentioned this issue May 16, 2022
Open
@tejas002 tejas002 mentioned this issue Oct 2, 2023
Open
@tejas002 tejas002 mentioned this issue Nov 27, 2023
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@FREEZX @amoshaviv @ilanbiala @s-oravec