Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blog article api - based on test-driven development #6

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
node_modules
.env
63 changes: 63 additions & 0 deletions app.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,63 @@
var express = require('express');
var path = require('path');
var logger = require('morgan');
var cookieParser = require('cookie-parser');
var bodyParser = require('body-parser');
var app = express();
var cors = require('cors')

var mongoose = require('mongoose');
mongoose.Promise = global.Promise;

var db_config = {
development: 'mongodb://localhost/article',
test: 'mongodb://localhost/article-test'
}
var current_env = app.settings.env

mongoose.connect(db_config[current_env], (err,res) => {
if(err) {
console.log(err);
} else {
console.log('connected to database: ', db_config[current_env]);
}
});

var index = require('./routes/index');
var articles = require('./routes/articles');
var users = require('./routes/users');

app.use(cors())
app.use(logger('dev'));
app.use(bodyParser.json());
app.use(bodyParser.urlencoded({ extended: false }));
app.use(cookieParser());
app.use(express.static(path.join(__dirname, 'public')));

app.use('/', index);
app.use('/api/articles', articles);
app.use('/api/users', users);

// catch 404 and forward to error handler
app.use(function(req, res, next) {
var err = new Error('Not Found');
err.status = 404;
next(err);
});

// error handler
app.use(function(err, req, res, next) {
// set locals, only providing error in development
res.locals.message = err.message;
res.locals.error = req.app.get('env') === 'development' ? err : {};

// render the error page
res.status(err.status || 500);
res.render('error');
});

app.listen(5000, ()=> {
console.log('server is listening at port 5000');
})

module.exports = app;
90 changes: 90 additions & 0 deletions bin/www
Original file line number Diff line number Diff line change
@@ -0,0 +1,90 @@
#!/usr/bin/env node

/**
* Module dependencies.
*/

var app = require('../app');
var debug = require('debug')('blog-tdd:server');
var http = require('http');

/**
* Get port from environment and store in Express.
*/

var port = normalizePort(process.env.PORT || '3000');
app.set('port', port);

/**
* Create HTTP server.
*/

var server = http.createServer(app);

/**
* Listen on provided port, on all network interfaces.
*/

server.listen(port);
server.on('error', onError);
server.on('listening', onListening);

/**
* Normalize a port into a number, string, or false.
*/

function normalizePort(val) {
var port = parseInt(val, 10);

if (isNaN(port)) {
// named pipe
return val;
}

if (port >= 0) {
// port number
return port;
}

return false;
}

/**
* Event listener for HTTP server "error" event.
*/

function onError(error) {
if (error.syscall !== 'listen') {
throw error;
}

var bind = typeof port === 'string'
? 'Pipe ' + port
: 'Port ' + port;

// handle specific listen errors with friendly messages
switch (error.code) {
case 'EACCES':
console.error(bind + ' requires elevated privileges');
process.exit(1);
break;
case 'EADDRINUSE':
console.error(bind + ' is already in use');
process.exit(1);
break;
default:
throw error;
}
}

/**
* Event listener for HTTP server "listening" event.
*/

function onListening() {
var addr = server.address();
var bind = typeof addr === 'string'
? 'pipe ' + addr
: 'port ' + addr.port;
debug('Listening on ' + bind);
}
50 changes: 50 additions & 0 deletions controllers/articleController.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,50 @@
var Article = require('../models/article')
var methods = {}

methods.getAll = (req, res) => {
Article.find({})
.then((articles)=>{
res.send(articles)
})
.catch((err)=>{res.send(err)})
}

methods.getByID = (req, res) => {
Article.findById(req.params.id)
.then((article)=>{
res.send(article)
})
.catch((err)=>{res.send(err)})
}

methods.create = (req, res) => {
Article.create(req.body)
.then(()=>{
res.send('Article added')
})
.catch((err)=>{res.send(err)})
}

methods.update = (req, res) => {
let id = req.params.id
Article.update({_id: id}, req.body)
.then(() => {
Article.findById(id)
.then((article)=>{
res.send(article)
})
.catch((err)=>{res.send(err)})
})
.catch((err)=>{res.send(err)})
}

methods.delete = (req, res) => {
let id = req.params.id
Article.deleteOne({_id: id})
.then(()=>{
res.send('Article removed')
})
.catch((err)=>{res.send(err)})
}

module.exports = methods
106 changes: 106 additions & 0 deletions controllers/userController.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,106 @@
require('dotenv').config();
const saltRounds = 10;

var User = require('../models/user');
var bcrypt = require('bcrypt');
const jwt = require('jsonwebtoken');
const secret = process.env.TOKEN_SECRET;
var methods = {}

methods.findAll = (req, res) => {
User.find({})
.then((users)=>{
res.send(users)
})
.catch(err => console.log(err))
}

methods.signup = (req, res) => {
var salt = bcrypt.genSaltSync(saltRounds);
var hash = bcrypt.hashSync(req.body.password, salt);

User.create({
username: req.body.username,
password: hash,
email: req.body.email,
createdAt: new Date()
})
.then(response => {
User.findById(response._id)
.then(user => {
res.send(user)
})
})
.catch(err => {
// console.log(err.message);
if(/username_1/.test(err.message)){
res.send('username already taken')
}
else if(/email_1/.test(err.message)){
res.send('email is already registered')
}
else {
res.send(err)
}
})
}

methods.login = (req, res) => {
var username = req.body.username;
var password = req.body.password;

User.findOne({username: username})
.then(user => {
if(user == null){
res.send('no such user')
} else {
bcrypt.compare(password, user.password)
.then(result => {
if(result) {
var token = jwt.sign({id: user.id, email: user.email, username: user.username}, secret);
res.send({'token': token});
} else {
res.send("password is incorrect");
}
})
.catch(err => console.log(err))
}
})
.catch(err => console.log(err))
}

methods.getOne = (req, res) => {
User.findById(req.params.id)
.then(user => {
res.send(user)
})
.catch(err => console.log(err))
}

methods.update = (req, res) => {
if(req.body.password){
var salt = bcrypt.genSaltSync(saltRounds);
var hash = bcrypt.hashSync(req.body.password, salt);
req.body.password = hash
}
User.findByIdAndUpdate(req.params.id, { $set: req.body }, { runValidators: true }, (err, user) => {
if(err) res.send(err.errors)
else {
User.findById(req.params.id)
.then(user => {
res.send(user)
})
.catch(err => console.log(err))
}
})
}

methods.remove = (req, res) => {
User.findOneAndRemove({_id: req.params.id})
.then (() => {
res.send('user has been deleted')
})
.catch(err => console.log(err))
}

module.exports = methods
14 changes: 14 additions & 0 deletions models/article.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
'use strict'
var mongoose = require('mongoose');
var Schema = mongoose.Schema;

var artcileSchema = new Schema({
title: String,
author: String,
content: String,
createdAt: Date
})

var Article = mongoose.model('Article', artcileSchema);

module.exports = Article;
32 changes: 32 additions & 0 deletions models/user.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
var mongoose = require('mongoose');
var Schema = mongoose.Schema;
const validator = require('validator');

var userSchema = new Schema({
username: {
type: String,
unique: [true, 'username is already used'],
required: [true, 'Please enter your name.'],
minlength: [3, 'Username must be at least 3 letters long.']
},
password: {
type: String,
required: [true, 'Please enter your password.']
},
email: {
type: String,
unique: [true, 'email is already used'],
required: [true, 'Please enter your email.'],
validate: {
validator: function(value) {
return validator.isEmail(value)
},
message: '{VALUE} is not a valid email.'
}
},
createdAt: Date
})

var User = mongoose.model('User', userSchema);

module.exports = User;
Loading