learn-open 1.2.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7330aa259179cd5f2c28ba41d279ac632266ba03
-  data.tar.gz: a70cf18a1f3791570d59a3185de77a2f9b572501
+  metadata.gz: 754f2bb0465443b07c6813b89bf718c97e01efbe
+  data.tar.gz: 846b8500f598af688e6162d078bbb64594024dd7
 SHA512:
-  metadata.gz: e3cdc0b4da3b50bb866ac3257b5ac8f6b723276eeaa1fa998342af876d9cf4bc866cfd5745a004b3f870285e7523f1c3d5af0a7cd250ebc3e9b4b9eb9c03fc8f
-  data.tar.gz: f6e7f8a06fa9feb544a8682defa64d5b139ed20d93a33c7559f87ff60ad3c171986f921b5fba666f82968f50d41281da8a533ecfac7a65eda5c82aceb4e7912e
+  metadata.gz: 7da69af22e33989482ee69b598f0fb91232f692aedb914d56d00591e50ae67612f3b42773e5752c470704142a1422ab83f278e9e2b773434d84b90f1020a3046
+  data.tar.gz: f4dbe515cc2b81abcaa9496d7f4f1674f177371dd359448dfafbe10a89d1e381d0b4193a28dcddf8f3d3bcb48fa265cee8cf13ce69bd57d8e4ee2af547bb3c96

data/lib/learn_open/opener.rb

@@ -1,5 +1,6 @@
 module LearnOpen
   class Opener
+    HOME_DIR = File.expand_path("~")
     attr_reader   :editor, :client, :lessons_dir, :file_path, :get_next_lesson, :token
     attr_accessor :lesson, :repo_dir, :lesson_is_lab, :lesson_id, :later_lesson, :dot_learn
 
@@ -14,8 +15,8 @@ module LearnOpen
       @lesson          = lesson
       @editor          = editor
       @get_next_lesson = get_next_lesson
-      @lessons_dir     = YAML.load(File.read(File.expand_path('~/.learn-config')))[:learn_directory]
-      @file_path       = File.expand_path('~/.learn-open-tmp')
+      @lessons_dir     = YAML.load(File.read("#{HOME_DIR}/.learn-config"))[:learn_directory]
+      @file_path       = "#{HOME_DIR}/.learn-open-tmp"
     end
 
     def run
@@ -37,6 +38,18 @@ module LearnOpen
       end
     end
 
+    def repo_exists?
+      done_labs = "#{HOME_DIR}/.done_labs"
+      if File.exists?(done_labs)
+        File.
+          readlines(done_labs).
+          grep(/#{repo_dir}/).
+          any?
+      else
+        File.exists?("#{lessons_dir}/#{repo_dir}")
+      end
+    end
+
     private
 
     def ping_fork_completion(retries=3)
@@ -248,10 +261,6 @@ module LearnOpen
       end
     end
 
-    def repo_exists?
-      File.exists?("#{lessons_dir}/#{repo_dir}")
-    end
-
     def open_with_editor
       if ios_lesson?
         open_ios_lesson

data/lib/learn_open/version.rb

@@ -1,3 +1,3 @@
 module LearnOpen
-  VERSION = '1.2.0'
+  VERSION = '1.2.1'
 end

data/spec/home_dir/.learn-config

@@ -0,0 +1,3 @@
+---
+:learn_directory: 'spec/home_dir/code/labs'
+:editor: 'atom'

data/spec/learn_open/opener_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+describe LearnOpen::Opener do
+  LearnOpen::Opener::HOME_DIR = File.join(__dir__, "..", "home_dir")
+
+  context "Verifying repo existance" do
+    let(:opener) { LearnOpen::Opener.new("","","") }
+    context "with .done_labs file" do
+      before do
+        done_labs = "#{LearnOpen::Opener::HOME_DIR}/.done_labs"
+        FileUtils.rm(done_labs) if File.exists?(done_labs)
+      end
+      it "returns true if lab is in the .done_labs file" do
+        expect(opener).to receive(:repo_dir).and_return("js-rubber-duck-wrangling")
+
+        File.open("#{LearnOpen::Opener::HOME_DIR}/.done_labs", "w+") do |f|
+          f.puts("js-rubber-duck-wrangling")
+        end
+
+        expect(opener.repo_exists?).to be_truthy
+      end
+      it "returns false if lab is not in the .done_labs file" do
+        expect(opener).to receive(:repo_dir).and_return("js-rubber-duck-wrangling")
+        expect(opener.repo_exists?).to be_falsy
+      end
+    end
+
+    context "without .done_labs file" do
+      before do
+        path = File.join(__dir__, "..", "home_dir", "code")
+        FileUtils.rm_rf(path)
+      end
+
+      it "returns true if directory for lab exists" do
+        expect(opener).to receive(:repo_dir).and_return("js-rubber-duck-wrangling")
+        FileUtils.mkdir_p("#{opener.lessons_dir}/js-rubber-duck-wrangling")
+
+        expect(opener.repo_exists?).to be_truthy
+      end
+
+      it "returns false if directory for lab doesn't exists" do
+        expect(opener).to receive(:repo_dir).and_return("js-rubber-duck-wrangling")
+        expect(opener.repo_exists?).to be_falsy
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1 @@
+require_relative '../lib/learn_open'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: learn-open
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors:
 - Flatiron School
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-14 00:00:00.000000000 Z
+date: 2016-09-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -99,6 +99,9 @@ files:
 - lib/learn_open/argument_parser.rb
 - lib/learn_open/opener.rb
 - lib/learn_open/version.rb
+- spec/home_dir/.learn-config
+- spec/learn_open/opener_spec.rb
+- spec/spec_helper.rb
 homepage: https://github.com/learn-co/learn-open
 licenses:
 - MIT
@@ -120,8 +123,11 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Open Learn lessons locally
-test_files: []
+test_files:
+- spec/home_dir/.learn-config
+- spec/learn_open/opener_spec.rb
+- spec/spec_helper.rb