procfile-upstart-exporter 0.0.1 → 0.0.2

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 2d2d1e53d73c1d9c485b125698e2e0ab41528de6
+  data.tar.gz: 4f13e9cb6eee0144dfb49ea5a000606b2403533d
+SHA512:
+  metadata.gz: 9ea702b920af27f119b7266aabc033b72ea4f809621fc2fc57c1f7aae53b6a806ab4b5fdd191bf1925ab19787e3878bab6d8056f823c969ce2c5082feb3afcc9
+  data.tar.gz: 343a676082a37552fee0982068e24233ac63dd64c8560b19e5ceddda96bc269e4e70c198e978e1c2c7f79f8221695d1c1807a73e299ae81275f5dcacc1fbbbff

data/.travis.yml

@@ -0,0 +1,5 @@
+---
+language: ruby
+rvm:
+  - '2.0.0'
+script: bundle exec rspec spec

data/README.md

@@ -50,7 +50,7 @@ Usage
 ### Create [Upstart][upstart] jobs configuration
 
 ```console
-# procfile-upstart-exporter export                                            \
+# procfile-upstart-exporter create                                            \
     --application <application>                                               \
     --procfile <path-to-procfile>                                             \
     --log <path-to-log>                                                       \
@@ -69,6 +69,27 @@ Usage
     --verbose
 ```
 
+Changelog
+---------
+
+### `v0.0.2`
+
+- Warn if `Procfile` or `.env` don't exist.
+- Treat properly spaces and comment lines in `.env`.
+
+### `v0.0.1`
+
+- First implementation of `craete` and `destroy` commands with basic
+  functionality.
+
+New release procedure
+---------------------
+
+1. Update changelog.
+2. Bump version.
+3. Push tag with version. For example, `v1.2.3`.
+4. Publish in [RubyGems][ruby-gem].
+
 License
 -------
 

data/lib/procfile_upstart_exporter/environment_parser.rb

@@ -3,10 +3,12 @@ class ProcfileUpstartExporter::EnvironmentParser
     ProcfileUpstartExporter.logger.debug "Start parsing environment file " \
                                          "`#{ environment }'"
     if File.exists? environment
-      File.read(environment).split("\n")
+      File.read(environment).split("\n").reject { |line|
+        line =~ /\A\s*(?:#.*)?\z/
+      }
     else
-      ProcfileUpstartExporter.logger.debug "Environment file " \
-                                           "`#{ environment }' does not exist"
+      ProcfileUpstartExporter.logger.warn "Environment file " \
+                                          "`#{ environment }' does not exist"
       []
     end
   end

data/lib/procfile_upstart_exporter/procfile_parser.rb

@@ -2,10 +2,16 @@ class ProcfileUpstartExporter::ProcfileParser
   def parse procfile
     ProcfileUpstartExporter.logger.debug 'Start parsing Procfile ' \
                                          "`#{ procfile }'"
-    File.read(procfile).split("\n").map do |process_line|
-      ProcfileUpstartExporter::Process.new(
-        *process_line.scan(/\A([^:]+):(.*)\z/).first.map(&:strip)
-      )
+    if File.exists? procfile
+      File.read(procfile).split("\n").map do |process_line|
+        ProcfileUpstartExporter::Process.new(
+          *process_line.scan(/\A([^:]+):(.*)\z/).first.map(&:strip)
+        )
+      end
+    else
+      ProcfileUpstartExporter.logger.warn "Procfile file " \
+                                          "`#{ procfile }' does not exist"
+      []
     end
   end
 end

data/lib/procfile_upstart_exporter/version.rb

@@ -1,3 +1,3 @@
 module ProcfileUpstartExporter
-  VERSION = '0.0.1'
+  VERSION = '0.0.2'
 end

data/procfile-upstart-exporter.gemspec

@@ -25,4 +25,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec', '~> 2.14'
   spec.add_development_dependency 'pry-nav'
+  spec.add_development_dependency 'coveralls'
 end

data/spec/fixtures/environment-with-spaces-and-comments

@@ -0,0 +1,8 @@
+# This is a comment, it should not be treated as environment variable.
+# The following line is empty, it should also be ignored:
+
+# The following line contains only spaces, thus should be ignored as well:
+    
+        # This is a comment that starts after a few spaces
+
+AN_ENVIRONMENT_VARIABLE=at_least

data/spec/procfile_upstart_exporter/creator_spec.rb

@@ -15,7 +15,7 @@ describe ProcfileUpstartExporter::Creator do
     let(:procfile)          { 'Procfile'                 }
     let(:log)               { "#{ temp_dir }/log"        }
     let(:application_log)   { "#{ log }/#{ application}" }
-    let(:environment)       { '.env'                     }
+    let(:environment)       { 'environment'              }
     let(:user)              { 'bin'                      }
     let(:upstart_jobs_path) { temp_dir                   }
     let(:application_root)  {
@@ -95,7 +95,7 @@ JOB_CONFIGURATION
       expect(File.directory? application_log).to be_true
     end
 
-    it "places environment variables from `.env' in process' Upstart job" do
+    it "places environment variables from environment file in Upstart job" do
       act
       expect(
         File.read("#{ upstart_jobs_path }/#{ application }/background.conf")

data/spec/procfile_upstart_exporter/environment_parser_spec.rb

@@ -8,7 +8,7 @@ describe ProcfileUpstartExporter::EnvironmentParser do
   describe '#parse' do
     subject(:environment_variables) { environment_parser.parse environment }
 
-    let(:environment) { 'spec/fixtures/sample-application/.env' }
+    let(:environment) { 'spec/fixtures/sample-application/environment' }
 
     it 'reads the environment file' do
       expect(File).to receive(:read).with(environment).and_call_original
@@ -28,5 +28,16 @@ describe ProcfileUpstartExporter::EnvironmentParser do
         expect(environment_variables).to eq([])
       end
     end
+
+    context 'environment file has empty lines and comments' do
+      let(:environment) {
+        'spec/fixtures/environment-with-spaces-and-comments'
+      }
+      it 'removes the empty lines and comments' do
+        expect(environment_variables).to eq(
+          ['AN_ENVIRONMENT_VARIABLE=at_least']
+        )
+      end
+    end
   end
 end

data/spec/procfile_upstart_exporter/procfile_parser_spec.rb

@@ -25,5 +25,13 @@ describe ProcfileUpstartExporter::ProcfileParser do
         )
       ])
     end
+
+    context 'inexistent Procfile' do
+      let(:procfile) { 'inexistent' }
+
+      it 'returns an empty Array' do
+        expect(processes).to eq([])
+      end
+    end
   end
 end

data/spec/spec_helper.rb

@@ -1,3 +1,6 @@
+require 'coveralls'
+Coveralls.wear!
+
 require_relative '../lib/procfile-upstart-exporter'
 
 require 'fileutils'

metadata

@@ -1,121 +1,124 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: procfile-upstart-exporter
-version: !ruby/object:Gem::Version 
-  hash: 29
-  prerelease: 
-  segments: 
-  - 0
-  - 0
-  - 1
-  version: 0.0.1
+version: !ruby/object:Gem::Version
+  version: 0.0.2
 platform: ruby
-authors: 
+authors:
 - Das Dad
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2013-10-17 00:00:00 +00:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2013-10-18 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: thor
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: values
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: values
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
-  name: bundler
   prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 9
-        segments: 
-        - 1
-        - 3
-        version: "1.3"
+      - !ruby/object:Gem::Version
+        version: '1.3'
   type: :development
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
-  name: rake
   prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
-  name: rspec
   prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 31
-        segments: 
-        - 2
-        - 14
-        version: "2.14"
+      - !ruby/object:Gem::Version
+        version: '2.14'
   type: :development
-  version_requirements: *id005
-- !ruby/object:Gem::Dependency 
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.14'
+- !ruby/object:Gem::Dependency
   name: pry-nav
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
   prerelease: false
-  requirement: &id006 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id006
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Export Procfile entries to Upstart jobs.
-email: 
+email:
 - dev@dasdad.com.br
-executables: 
+executables:
 - procfile-upstart-exporter
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
+files:
 - .gitignore
 - .rspec
+- .travis.yml
 - Gemfile
 - README.md
 - Rakefile
@@ -131,8 +134,9 @@ files:
 - lib/procfile_upstart_exporter/procfile_parser.rb
 - lib/procfile_upstart_exporter/version.rb
 - procfile-upstart-exporter.gemspec
-- spec/fixtures/sample-application/.env
+- spec/fixtures/environment-with-spaces-and-comments
 - spec/fixtures/sample-application/Procfile
+- spec/fixtures/sample-application/environment
 - spec/procfile_upstart_exporter/creator_spec.rb
 - spec/procfile_upstart_exporter/destroyer_spec.rb
 - spec/procfile_upstart_exporter/environment_parser_spec.rb
@@ -142,43 +146,34 @@ files:
 - spec/support/temp_dir.rb
 - templates/application.conf
 - templates/process.conf.erb
-has_rdoc: true
 homepage: https://github.com/dasdad/procfile-upstart-exporter
-licenses: 
+licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.6.2
+rubygems_version: 2.0.3
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Solve common problems with `foreman export`
-test_files: 
-- spec/fixtures/sample-application/.env
+test_files:
+- spec/fixtures/environment-with-spaces-and-comments
 - spec/fixtures/sample-application/Procfile
+- spec/fixtures/sample-application/environment
 - spec/procfile_upstart_exporter/creator_spec.rb
 - spec/procfile_upstart_exporter/destroyer_spec.rb
 - spec/procfile_upstart_exporter/environment_parser_spec.rb