Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

Python Build a Social Network with Flask Tacocat Challenge The Challenge

Nthulane Makgato
PLUS
Nthulane Makgato
Courses Plus Student 19,602 Points

Tests are still failing. Please help!

Hi there!

A few days ago I posted a help question about the Tacocat challenge(https://teamtreehouse.com/community/failing-a-few-tests-please-help). I had 5 tests that were not passing, I didn't get a response from the community. Managed to resolve 3 of them and have two left. Please help.

The challanges that aren't passing are:

  1. "test_taco_create" test. When I create a taco, it creates however it still fails when tested.
  2. "test_logged_out_menu" test. This is really wierd because its basically the
<nav class="menu">
            <!-- menu goes here -->
              {% if current_user.is_authenticated %}
                <a href="{{ url_for('logout') }}" title="Log Out">Logout</a>
                <a href="{{ url_for('taco') }}" title="New Taco">Add a New Taco</a>
              {% else %}
                <a href="{{ url_for('login') }}" title ="Log in">Login</a>
                <a href="{{ url_for('register') }}" title="Sign Up">Sign Up</a>
              {% endif %}
          </nav>

The "{% else %}" block of the code does not run. It could be that the user is just always logged in but when I ran "{% if not current_user.is_authenticated %}", then the user was always logged out and the "test_logged_in_menu" fails. The problem might just be with the "{% if %}{% else %}" statements.

tacocat.py

from flask import Flask, g, render_template, flash, redirect, url_for
from flask.ext.bcrypt import check_password_hash
from flask.ext.login import (LoginManager, login_user,
                             logout_user, login_required, current_user)

import forms
import models

DEBUG = True
PORT = 8000
HOST = '0.0.0.0'

app = Flask(__name__)
app.secret_key = 'youdontknowmysecretkey'

login_manager = LoginManager()
login_manager.init_app(app)
login_manager.login_view = 'login'

@login_manager.user_loader
def load_user(userid):
  try:
      return models.User.get(models.User.id == userid)
  except models.DoesNotExist:
      return None

@app.before_request
def before_request():
  """Connect to the database before each request."""
  g.db = models.DATABASE
  g.db.connect()
  g.user = current_user

@app.after_request
def after_request(response):
  """Close the database connection after each request."""
  g.db.close
  return response

@app.route('/register', methods=('GET', 'POST'))
def register():
  form = forms.RegisterForm()
  if form.validate_on_submit():
      flash("Yay, you registered!", "success")
      models.User.create_user(
          email = form.email.data,
          password = form.password.data
      )
      return redirect(url_for('index'))
  return render_template('register.html', form=form)

@app.route('/login', methods=('GET', 'POST'))
def login():
  form = forms.LoginForm()
  if form.validate_on_submit():
    try:
        user = models.User.get(models.User.email == form.email.data)
    except models.DoesNotExist:
        flash("Email is incompatible", "error")
    else:
        if check_password_hash(user.password, form.password.data):
            login_user(user)
            flash("You are logged in", "success")
            return redirect(url_for('index'))
        else:
            flash("Password does not match", "error")
  return render_template('login.html', form=form)

@app.route('/logout')
@login_required
def logout():
    logout_user()
    flash("You've been logged out! Come back soon!", "success")
    return redirect(url_for('index'))

@app.route('/new_taco', methods=('GET','POST'))
@login_required
def taco():
    form = forms.TacoForm()
    if form.validate_on_submit():
        models.Taco.create(user = g.user._get_current_object(),
                           protein = form.protein.data,
                           shell = form.shell.data,
                           cheese = form.cheese.data,
                           extras = form.extras.data.strip()
                          )
        flash("Your taco has been added!!", "success")
        return redirect(url_for('index'))
    return render_template('taco.html', form=form)


@app.route('/')
def index():
  tacos = models.Taco.select().limit(10)
  return render_template('index.html', tacos=tacos)

if __name__== '__main__':
  models.initialize()
  try:
    with models.DATABASE.transaction():
        models.User.create_user(
          email = 'billy@example.com',
          password = 'password'
        )
  except ValueError:
    pass
  app.run(debug=DEBUG, host=HOST, port=PORT)

forms.py

from flask_wtf import Form
from wtforms import StringField, PasswordField, BooleanField, TextAreaField
from wtforms.validators import (DataRequired, ValidationError, Email,
                               Length, EqualTo)

from models import User

def email_exists(form, field):
    if User.select().where(User.email == field.data).exists():
        raise ValidatationError('User with that email already exists')

class RegisterForm(Form):
    email = StringField(
        'Email',
        validators = [
            DataRequired(),
            Email(),
            email_exists,
      ])
    password = PasswordField(
        'Password',
        validators=[
            DataRequired(),
            Length(min=2),
            EqualTo('password2', message='Passwords must match')
      ])
    password2 = PasswordField(
        'Confirm Password',
        validators = [DataRequired()]
    )

class LoginForm(Form):
    email = StringField('Email', validators = [DataRequired(), Email()])
    password = PasswordField('Password', validators = [DataRequired()])

class TacoForm(Form):
    protein = StringField('Protein')
    shell = StringField('Shell', validators = [DataRequired()])
    cheese = BooleanField('Cheese')
    extras = TextAreaField("Any extras?")

models.py

import datetime

from flask.ext.bcrypt import generate_password_hash
from flask.ext.login import UserMixin
from peewee import *

DATABASE = SqliteDatabase('taco.db')

class User(UserMixin, Model):
    email = CharField(unique=True)
    password = CharField(max_length=100)

    class Meta:
      database = DATABASE

    def get_tacos(self):
      return Taco.select().where(Taco.user == self)

    def get_stream(self):
      return Taco.select().where(
          (Taco.user == self)
      )



    @classmethod
    def create_user(cls, email, password):
      try:
        cls.create(
          email = email,
          password = generate_password_hash(password))
      except IntegrityError:
        raise ValueError("User already exists")

class Taco(Model):
    protein = CharField()
    shell = CharField()
    cheese = BooleanField(default=True)
    extras = TextField()
    user = ForeignKeyField(
        rel_model = User,
        related_name = 'tacos'
    )

    class Meta:
      database = DATABASE


def initialize():
  DATABASE.connect()
  DATABASE.create_tables([User, Taco], safe=True)
  DATABASE.close()

layout.html

<!doctype html>
<html>
  <head>
    <title>Tacocat</title>
    <link rel="stylesheet" href="/static/css/normalize.css">
    <link rel="stylesheet" href="/static/css/skeleton.css">
    <link rel="stylesheet" href="/static/css/tacocat.css">
  </head>
  <body>
    {% with messages=get_flashed_messages() %}
    {% if messages %}
        <div class="messages">
          {% for message in messages %}
          <div class="message">
            {{ message }}
          </div>
          {% endfor %}
        </div>
    {% endif %}
    {% endwith %}

    <div class="container">
      <div class="row">
        <div class="u-full-width">
          <nav class="menu">
            <!-- menu goes here -->
              {% if current_user.is_authenticated %}
                <a href="{{ url_for('logout') }}" title="Log Out">Logout</a>
                <a href="{{ url_for('taco') }}" title="New Taco">Add a New Taco</a>
              {% else %}
                <a href="{{ url_for('login') }}" title ="Log in">Login</a>
                <a href="{{ url_for('register') }}" title="Sign Up">Sign Up</a>
              {% endif %}
          </nav>
          {% block content %}{% endblock %}
        </div>
      </div>
      <footer>
        <p>An MVP web app made with Flask on <a href="http://teamtreehouse.com">Treehouse</a>.</p>
      </footer>
    </div>
  </body>
</html>

Thanks in advance for your help!!

4 Answers

Chris Freeman
MOD
Chris Freeman
Treehouse Moderator 68,441 Points

Partial answer for test_logged_out_menu: Sometimes adding parens fixes it.

{# change #}
    {% if current_user.is_authenticated %}

{# to #}
    {% if current_user.is_authenticated() %}

Post back if this works.

I'll look into the other issue later this weekend.

Nthulane Makgato
PLUS
Nthulane Makgato
Courses Plus Student 19,602 Points

Hi Chris Freeman, I managed to sort out the last issue with the tests.

#app_tests.py wanted:
@app.route('/taco', methods=('GET','POST'))

#instead of:
@app.route('/new_taco', methods=('GET','POST'))

The code now passes all the tests in workspace but not the challenge. It just says "bummer, try again". Its not related to app.run() as I have removed that. I will try it in a few hours and if it persists then I will pose a new question.

Thanks for all the help ;)

Nthulane Makgato
PLUS
Nthulane Makgato
Courses Plus Student 19,602 Points

Wow... it worked. I wonder if it should because of the update that states no parenthesis. Thanks.

I look forward to your input on the last failure ;)