bryanaknight

This PR is to spike out an implementation of including tenant_id when an email notification is generated so when we verify the generated and encoded token, we are connected to the correct tenant schema. We are proposing a globally available load_tenant!(tenant) method that is responsible for setting the tenant schema context, as opposed to passing a tenant_id throughout the entire mail reply code path.

###Questions

How do we support existing signed auth tokens that do not include a tenant_id?

An approach that we discussed is to add a new PACK_FORMAT to use for newly generated tokens (those that include tenant_id going forward) so we can differentiate between pre-tenant and post-tenant tokens. This may warrant new GitHub::Authentication::SignedAuthToken version. Pre-tenant tokens will signal to us that the applicable tenant is GitHub, and for post-tenant tokens we'll parse and retrieve the tenant_id from the token. We can then connect to the appopriate tenant schema.

bryanaknight

module GitHub
  class Cache

    def fetch(*opts)
      opts[:cache_key].prepend current_tenant.id
      super(opts)
    end

    def get(*opts)
      opts[:cache_key].prepend current_tenant.id

bryanaknight

  1) Failure:
Api::SearchTest#test__search_issues_sort_issues_in_ascending_order_L936 [/Users/bryanaknight/github/github/test/integration/api/search_test.rb:941]:
Expected: 2
  Actual: 0

  2) Failure:
Api::SearchTest#test__search_issues_include_issues_in_private_repo_for_integration_bot_with_access_to_the_repo_s_issues__with_search_scoped_to_that_repo_L853 [/Users/bryanaknight/github/github/test/integ$
ation/api/search_test.rb:865]:
Expected: 1
  Actual: 0

bryanaknight

runtime! autoload/pathogen.vim

syntax on
filetype plugin indent on

set visualbell

set wildmenu
set wildmode=list:longest,full

bryanaknight

require 'oauth'
require 'json'
require 'pp'

  CONSUMER_KEY = OAuth::Consumer.new(
    'qLqLjhVG6TQDesSYnT3bw',
    '9Zw1OIB17651BMZUEsRlA5CKk3qG1MJclzr0f0Qq0')

  ACCESS_TOKEN = OAuth::Token.new(
    '321130543-9SPLOfIzXbHDuWnaZSyBBm3ctix0xrh6REooGZSd',

bryanaknight

  describe "adding items and viewing cart" do 

    it "has unique cart for each restaurant" do
      new_restaurant = FactoryGirl.create(:restaurant)
      new_item = FactoryGirl.create(:item, restaurant_id: new_restaurant.id)
      visit "/#{new_restaurant.slug}"
      click_on('Add to Cart')
      expect(page).to have_content("Bacon")
      new_restaurant2 = FactoryGirl.create(:restaurant, title: "Parsley", description: "blah balh")
      new_item2 = FactoryGirl.create(:item, restaurant_id: new_restaurant2.id, title: "brown fried food", description: "fried brown food")

bryanaknight

class Hamming

  def self.compute(sequence1, sequence2)
    sequence1 = sequence1.split('')
    sequence2 = sequence2.split('')
    sequence1.zip(sequence2).count do |string_a, string_b| 
      self.substitution?(string_a, string_b)
    end
  end

bryanaknight

return true if number.even?
  else
return false
 
vs. 
 
def test_count_one_word
  phrase = Phrase.new("word")
  counts = {"word" => 1}
  assert_equal counts, phrase.word_count

bryanaknight

require 'test_helper'

class ArticlesControllerTest < ActionController::TestCase

  test "index method lists all articles" do
    Article.create(title: "First", body: "Body")
    Article.create(title: "Second", body: "Body")
    articles = Article.all
    assert_equal 2, articles.count
    #when I run rake, getting 4 articles instead of 2

bryanaknight

class Scrabble

  def initialize
    SCORES = {"A"=>1, "B"=>3, "C"=>3, "D"=>2, "E"=>1, "F"=>4, "G"=>2, "H"=>4, "I"=>1, "J"=>8,
    "K"=>5, "L"=>1, "M"=>3, "N"=>1, "O"=>1, "P"=>3, "Q"=>10, "R"=>1, "S"=>1, "T"=>1,
    "U"=>1, "V"=>4, "W"=>4, "X"=>8, "Y"=>4, "Z"=>10}
  end
  def self.score(word)
    SCORES[word.upcase]
  end