Error user count

I’m sure there is something wrong in my code, but I cannot be sure. I’ve poured over it several times and I’m just missing something. Getting this error:

Failures:

  1) Signing up allows a user to sign up for the site and creates the object in the database
     Failure/Error: expect(User.count).to eq(1)

       expected: 1
            got: 0

       (compared using ==)
     # ./spec/features/users/registration_spec.rb:18:in `block (2 levels) in <top (required)>'

registration_spec.rb

require "spec_helper"

describe "Signing up" do
    it "allows a user to sign up for the site and creates the object in the database" do
        expect(User.count).to eq(0)

        visit "/"
        expect(page).to have_content("Sign Up")
        click_link "Sign Up"

        fill_in "First Name", with: "Jason"
        fill_in "Last Name", with: "Seifer"
        fill_in "Email", with: "jason@teamtreehouse.com"
        fill_in "Password", with:" treehouse1234"
        fill_in "Password (again)", with: "treehouse1234"
        click_button "Sign Up"

        expect(User.count).to eq(1)
    end
end

_form.html.erb

<%= form_for(@user) do |f| %>
  <% if @user.errors.any? %>
    <div id="error_explanation">
      <h2><%= pluralize(@user.errors.count, "error") %> prohibited this user from being saved:</h2>

      <ul>
      <% @user.errors.full_messages.each do |message| %>
        <li><%= message %></li>
      <% end %>
      </ul>
    </div>
  <% end %>

  <div class="field">
    <%= f.label :first_name, "First Name" %><br>
    <%= f.text_field :first_name %>
  </div>
  <div class="field">
    <%= f.label :last_name, "Last Name" %><br>
    <%= f.text_field :last_name %>
  </div>
  <div class="field">
    <%= f.label :email %><br>
    <%= f.text_field :email %>
  </div>
  <div class="field">
    <%= f.label :password %><br>
    <%= f.password_field :password %>
  </div>
  <div class="field">
    <%= f.label :password_confirmation, "Password (again)" %><br>
    <%= f.password_field :password_confirmation %>
  </div>
  <div class="actions">
    <%= f.submit "Sign Up" %>
  </div>
<% end %>

in the above section the first two and last two lines are being removed by the code block and I do not know why. I assure you they’re there.

users_controller.rb

class UsersController < ApplicationController
  before_action :set_user, only: [:show, :edit, :update, :destroy]

  # GET /users/new
  def new
    @user = User.new
  end

  # GET /users/1/edit
  def edit
  end

  # POST /users
  # POST /users.json
  def create
    @user = User.new(user_params)

    respond_to do |format|
      if @user.save
        format.html { redirect_to @user, notice: 'User was successfully created.' }
        format.json { render :show, status: :created, location: @user }
      else
        format.html { render :new }
        format.json { render json: @user.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /users/1
  # PATCH/PUT /users/1.json
  def update
    respond_to do |format|
      if @user.update(user_params)
        format.html { redirect_to @user, notice: 'User was successfully updated.' }
        format.json { render :show, status: :ok, location: @user }
      else
        format.html { render :edit }
        format.json { render json: @user.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /users/1
  # DELETE /users/1.json
  def destroy
    @user.destroy
    respond_to do |format|
      format.html { redirect_to users_url, notice: 'User was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_user
      @user = User.find(params[:id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def user_params
      params.require(:user).permit(:first_name, :last_name, :email, :password, :password_confirmation)
    end
end

If you require any additional information I will provide it.

• Mark as New
• Bookmark Message
• Subscribe to Message
• Mute Message
• Subscribe to RSS Feed
• Permalink
• Print
• Report Inappropriate Content

New answer!

OK. You need rows of error, columns of pages, and each cell to say «X/Y», for example «9/2131» where it says that 9 users experienced that error on that particular page, and 2131 total users visited that page.

Here you go:

| stats values(ipAddress) as addresses dc(ipAddress) as impact by error Page
 | streamstats dc(addresses) as totalUsersImpacted by Page
 | fields - addresses
 | eval impact=impact + "/" + totalUsersImpacted
 | xyseries error Page impact

You need to do dc once in the regular stats command and then a second time with streamstats … by error. This streamstats clause will do the same dc math but only per Error. To give the streamstats the raw materials we need, we need to pass along the values(ipAddress) although after streamstats we can safely discard them.

• Mark as New
• Bookmark Message
• Subscribe to Message
• Mute Message
• Subscribe to RSS Feed
• Permalink
• Print
• Report Inappropriate Content

If you have fields userId, page and error, it would look like this:

<your search terms> | chart dc(userId) over error by page limit=100

The limit=100 is because by default the chart command will display 9 split by values and then roll all the rest up into a big one called «OTHER». limit=100 raises that threshold from 10 to 100.

• Mark as New
• Bookmark Message
• Subscribe to Message
• Mute Message
• Subscribe to RSS Feed
• Permalink
• Print
• Report Inappropriate Content

In other words I have a query that currently looks like this and works
search here
| stats dc(ipAddress) AS impact by Page error
| xyseries error,Page ,impact
| fillnull value=0

The psuedo code of what I want is
search here
| stats dc(ipAddressErrors).»/».dc(totalIpAddresses) AS impact by Page error
| xyseries error,Page ,impact
| fillnull value=0

• Mark as New
• Bookmark Message
• Subscribe to Message
• Mute Message
• Subscribe to RSS Feed
• Permalink
• Print
• Report Inappropriate Content

Understood. You need rows of error, columns of pages, and each cell to say «9/2131» meaning that 9 users experienced that error on that particular page, and 2131 total users experienced that error across all pages.

Then what you need will look like this.

| stats values(ipAddress) as addresses dc(ipAddress) as impact by error Page
| streamstats dc(addresses) as totalUsersImpacted by error
| fields - addresses
| eval impact=impact + "/" + totalUsersImpacted
| xyseries error Page impact

You need to do dc once in the regular stats command and then a second time with streamstats … by error. This streamstats clause will do the same dc math but only per error. To give the streamstats the raw materials we need, we need to pass along the values(ipAddress) although after streamstats we can safely discard them.

• Mark as New
• Bookmark Message
• Subscribe to Message
• Mute Message
• Subscribe to RSS Feed
• Permalink
• Print
• Report Inappropriate Content

close, but this part
«Understood. You need rows of error, columns of pages, and each cell to say «9/2131″ meaning that 9 users experienced that error on that particular page, and 2131 total users experienced that error across all pages.»

should read
«Understood. You need rows of error, columns of pages, and each cell to say «9/2131» meaning that 9 users experienced that error on that particular page, and 2131 total users visited that page.»

• Mark as New
• Bookmark Message
• Subscribe to Message
• Mute Message
• Subscribe to RSS Feed
• Permalink
• Print
• Report Inappropriate Content

Ah, easy enough.

| stats values(ipAddress) as addresses dc(ipAddress) as impact by error Page
| streamstats dc(addresses) as totalUsersImpacted by Page
| fields - addresses
| eval impact=impact + "/" + totalUsersImpacted
| xyseries error Page impact

I’ll post it as a separate answer since this one began pretty far away from what you needed.

• Mark as New
• Bookmark Message
• Subscribe to Message
• Mute Message
• Subscribe to RSS Feed
• Permalink
• Print
• Report Inappropriate Content

Hmm, this still is not doing what I would like, When searching back 60 minutes, it will report data like this
Logon Search
404 1/35 2/23

The problem is, the value for the 1 and 2 seems correct, but the second number is way off, if I do a
….page=logon
|top 0 ipAddress

I get say 157, so I would expect it to show 1/157 for that value.

Where there were 157 unique ipAddress values for logon that did or did not receive the error, and 1 that did during the search time frame.

• Mark as New
• Bookmark Message
• Subscribe to Message
• Mute Message
• Subscribe to RSS Feed
• Permalink
• Print
• Report Inappropriate Content

Sounds like either Page or error is undefined for a lot of your requests. I assumed you didn’t want to count such events.

My first stats command there will implicitly discard rows that don’t have a value for either error or Page. If you want to include counts for things that don’t have error or Page values, then add before the first stats, | fillnull error Page value="NULL"

• Mark as New
• Bookmark Message
• Subscribe to Message
• Mute Message
• Subscribe to RSS Feed
• Permalink
• Print
• Report Inappropriate Content

I see what my issue here is based on your good explanation. My issue is, the field that I am regex the error out of the event doesn’t exist at all in the non-error events, so filling null, effectively doesn’t do anything. I need to figure out a way to include the total count. I have another field that is common between both events, error and non-error. LogLevel=INFO or LogLevel=ERROR. So I need to figure out a way to get the total unique non-error+error users now.