Revert "IE 1122 update deprecated jira apis" (#25)

* Revert "IE 1122 update deprecated jira apis (#24)"

This reverts commit aa6813f.

* github action updated in rspec

Author: aditi-kantata

README.rdoc

@@ -235,7 +235,6 @@ Here's the same example as a Sinatra application:
         :access_token_path  => "/plugins/servlet/oauth/access-token",
         :private_key_file   => "rsakey.pem",
         :rest_base_path     => "/rest/api/2",
-        :rest_base_path_v3  => "/rest/api/3",
         :consumer_key       => "jira-ruby-example"
       }
 

lib/jira/client.rb

@@ -50,7 +50,6 @@ class Client
       :site               => 'http://localhost:2990',
       :context_path       => '/jira',
       :rest_base_path     => "/rest/api/2",
-      :rest_base_path_v3 => "/rest/api/3",
       :ssl_verify_mode    => OpenSSL::SSL::VERIFY_PEER,
       :use_ssl            => true,
       :auth_type          => :oauth,
@@ -61,7 +60,6 @@ def initialize(options={})
       options = DEFAULT_OPTIONS.merge(options)
       @options = options
       @options[:rest_base_path] = @options[:context_path] + @options[:rest_base_path]
-      @options[:rest_base_path_v3] = @options[:context_path] + @options[:rest_base_path_v3]
 
       case options[:auth_type]
       when :oauth

lib/jira/resource/issue.rb

@@ -47,16 +47,15 @@ class Issue < JIRA::Base
       has_many :remotelink, :class => JIRA::Resource::Remotelink
 
       def self.all(client)
-        url = client.options[:rest_base_path_v3] + '/search/jql?expand=transitions.fields'
+        url = client.options[:rest_base_path] + "/search?expand=transitions.fields"
         response = client.get(url)
         json = parse_json(response.body)
         json['issues'].map do |issue|
           client.Issue.build(issue)
         end
       end
 
-
-      def self.jql_v2(client, jql, options = {fields: nil, start_at: nil, max_results: nil, expand: nil})
+      def self.jql(client, jql, options = {fields: nil, start_at: nil, max_results: nil, expand: nil})
         url = client.options[:rest_base_path] + "/search?jql=" + CGI.escape(jql)
 
         url << "&fields=#{options[:fields].map{ |value| CGI.escape(client.Field.name_to_id(value)) }.join(',')}" if options[:fields]
@@ -75,29 +74,6 @@ def self.jql_v2(client, jql, options = {fields: nil, start_at: nil, max_results:
         end
       end
 
-      def self.jql(client, jql, options = {fields: nil, next_page_token: nil, max_results: nil, expand: nil})
-        url = client.options[:rest_base_path_v3] + "/search/jql?jql=" + CGI.escape(jql)
-
-        url << "&fields=#{options[:fields].map{ |value| CGI.escape(client.Field.name_to_id(value)) }.join(',')}" if options[:fields]
-        url << "&nextPageToken=#{CGI.escape(options[:next_page_token].to_s)}" if options[:next_page_token]
-        url << "&maxResults=#{CGI.escape(options[:max_results].to_s)}" if options[:max_results]
-
-        if options[:expand]
-          options[:expand] = [options[:expand]] if options[:expand].is_a?(String)
-          url << "&expand=#{options[:expand].to_a.map{ |value| CGI.escape(value.to_s) }.join(',')}"
-        end
-
-        response = client.get(url)
-        json = parse_json(response.body)
-        result = {}
-        result['next_page_token'] = json['nextPageToken'] if json['nextPageToken'] rescue nil
-        result['issues'] = json['issues'].map do |issue|
-          client.Issue.build(issue)
-        
-        end
-        result
-      end
-
       def editmeta
         editmeta_url = client.options[:rest_base_path] + "/#{self.class.endpoint_name}/#{key}/editmeta"
 

lib/jira/resource/project.rb

@@ -17,7 +17,7 @@ def self.key_attribute
 
       # Returns all the issues for this project
       def issues(options={})
-        search_url = client.options[:rest_base_path_v3] + "/search/jql"
+        search_url = client.options[:rest_base_path] + '/search'
         query_params = {:jql => "project=\"#{key}\""}
         query_params.update Base.query_params_for_search(options)
         response = client.get(url_with_query_params(search_url, query_params))

lib/jira/resource/rapidview.rb

@@ -32,9 +32,9 @@ def issues(options = {})
         jql = "id IN(#{issue_ids.join(', ')})"
 
         # Filtering options
-        jql << " AND sprint IS NOT EMPTY" unless options[:include_backlog_items] 
+        jql << " AND sprint IS NOT EMPTY" unless options[:include_backlog_items]
 
-        parent_issues = client.Issue.jql(jql)["issues"]
+        parent_issues = client.Issue.jql(jql)
         subtask_ids = parent_issues.map { |t| t.subtasks.map { |sub| sub['id'] } }.flatten
 
         parent_and_sub_ids = parent_issues.map(&:id) + subtask_ids

spec/integration/comment_spec.rb

@@ -42,15 +42,7 @@
     let(:expected_attributes_from_put) {
       { "id" => "10000", "body" => "new body" }
     }
-    before(:each) do
-      stub_request(:get, "http://foo:bar@localhost:2990/jira/rest/api/2/comment")
-        .with(headers: {
-          'Accept'=>'application/json',
-          'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
-          'User-Agent'=>'Ruby'
-        })
-        .to_return(:status => 200, :body => '{"comments":[{"self":"http://localhost:2990/jira/rest/api/2/issue/10002/comment/10000","id":"10000","body":"This is a comment. Creative."},{"self":"http://localhost:2990/jira/rest/api/2/issue/10002/comment/10001","id":"10001","body":"Another comment."}]}', :headers => {})
-    end
+
     it_should_behave_like "a resource"
     it_should_behave_like "a resource with a collection GET endpoint"
     it_should_behave_like "a resource with a singular GET endpoint"

spec/integration/field_spec.rb

@@ -26,15 +26,7 @@
     end
 
     let(:expected_collection_length) { 2 }
-    before(:each) do
-      stub_request(:get, "http://foo:bar@localhost:2990/jira/rest/api/2/field")
-        .with(headers: {
-          'Accept'=>'application/json',
-          'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
-          'User-Agent'=>'Ruby'
-        })
-        .to_return(:status => 200, :body => '[{"id":"1","name":"Description","custom":false,"orderable":true,"navigable":true,"searchable":true,"clauseNames":["description"],"schema":{"type":"string","system":"description"}},{"id":"2","name":"Summary","custom":false,"orderable":true,"navigable":true,"searchable":true,"clauseNames":["summary"],"schema":{"type":"string","system":"summary"}}]', :headers => {})
-    end
+
     it_should_behave_like "a resource"
     it_should_behave_like "a resource with a collection GET endpoint"
     it_should_behave_like "a resource with a singular GET endpoint"

spec/integration/issue_spec.rb

@@ -46,7 +46,7 @@
         }
       }
       before(:each) do
-        stub_request(:get, site_url + "/jira/rest/api/3/search/jql?expand=transitions.fields").
+        stub_request(:get, site_url + "/jira/rest/api/2/search?expand=transitions.fields").
                     to_return(:status => 200, :body => get_mock_response('issue.json'))
       end
       it_should_behave_like "a resource with a collection GET endpoint"
@@ -55,8 +55,8 @@
     it_should_behave_like "a resource with a POST endpoint"
     it_should_behave_like "a resource with a PUT endpoint"
     it_should_behave_like "a resource with a PUT endpoint that rejects invalid fields"
+
     describe "errors" do
-      
       before(:each) do
         stub_request(:get,
                     site_url + "/jira/rest/api/2/issue/10002").
@@ -87,25 +87,6 @@
           "key"=>"SAMPLEPROJECT-13"
         }
       }
-      
-      before(:each) do
-        
-        stub_request(:get, "http://foo:bar@localhost:2990/jira/rest/api/3/search/jql?jql=PROJECT%20=%20'SAMPLEPROJECT'")
-          .with(headers: {
-            'Accept'=>'application/json',
-            'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
-            'User-Agent'=>'Ruby'
-          })
-          .to_return(status: 200, body: get_mock_response('issue.json'), headers: {})
-        stub_request(:get, site_url + "/jira/rest/api/3/search/jql?jql=PROJECT%20=%20'SAMPLEPROJECT'")
-          .with(headers: {
-            'Accept'=>'application/json',
-            'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
-            'Authorization'=>/OAuth .*/, # Use a regex to match any OAuth header
-            'User-Agent'=>'OAuth gem v0.5.14'
-    })
-          .to_return(status: 200, body: get_mock_response('issue.json'), headers: {})
-        end
       it_should_behave_like "a resource with JQL inputs and a collection GET endpoint"
     end
 

spec/integration/issuelinktype_spec.rb

@@ -2,20 +2,6 @@
 
 describe JIRA::Resource::Issuelinktype do
 
-  before(:each) do
-    stub_request(:get, "http://foo:bar@localhost:2990/jira/rest/api/2/issueLinkType")
-      .with(headers: {
-        'Accept'=>'application/json',
-        'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
-        'User-Agent'=>'Ruby'
-      })
-      .to_return(
-        :status => 200,
-        :body => '{"issueLinkTypes":[{"id":"10000","self":"http://localhost:2990/jira/rest/api/2/issueLinkType/10000","name":"Blocks","inward":"is blocked by","outward":"blocks"},{"id":"10001","self":"http://localhost:2990/jira/rest/api/2/issueLinkType/10001","name":"Relates","inward":"relates to","outward":"relates to"},{"id":"10002","self":"http://localhost:2990/jira/rest/api/2/issueLinkType/10002","name":"Duplicates","inward":"is duplicated by","outward":"duplicates"}]}',
-        :headers => {}
-      )
-  end
-
   with_each_client do |site_url, client|
     let(:client) { client }
     let(:site_url) { site_url }

spec/integration/issuetype_spec.rb

@@ -18,16 +18,6 @@
 
     let(:expected_collection_length) { 5 }
 
-    before(:each) do
-      stub_request(:get, "http://foo:bar@localhost:2990/jira/rest/api/2/issuetype")
-        .with(headers: {
-          'Accept'=>'application/json',
-          'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
-          'User-Agent'=>'Ruby'
-        })
-        .to_return(:status => 200, :body => '[{"self":"http://localhost:2990/jira/rest/api/2/issuetype/5","id":"5","name":"Sub-task"},{"self":"http://localhost:2990/jira/rest/api/2/issuetype/1","id":"1","name":"Bug"},{"self":"http://localhost:2990/jira/rest/api/2/issuetype/2","id":"2","name":"Task"},{"self":"http://localhost:2990/jira/rest/api/2/issuetype/3","id":"3","name":"Story"},{"self":"http://localhost:2990/jira/rest/api/2/issuetype/4","id":"4","name":"Epic"}]', :headers => {})
-    end
-
     it_should_behave_like "a resource"
     it_should_behave_like "a resource with a collection GET endpoint"
     it_should_behave_like "a resource with a singular GET endpoint"