File obs-server-2.8-0107-ci-Rails-5.1-Fix-test-failu... of Package obs-server

Overview Repositories Revisions Requests Users Attributes Meta

File obs-server-2.8-0107-ci-Rails-5.1-Fix-test-failure-caused-by-missing-requ.patch of Package obs-server (Revision 617c434d5a327d4f9d9c86ae16dc58d1)

Currently displaying revision 617c434d5a327d4f9d9c86ae16dc58d1 , Show latest

From 9b7ac487939c2f9eb67a8acac0b8320f2b95363d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Geuken?= <bgeuken@suse.de>
Date: Fri, 16 Jun 2017 14:54:12 +0200
Subject: [PATCH] [ci] Rails 5.1: Fix test failure caused by missing require

===

1) Webui::PatchinfoController POST #save when the patchinfo's xml is invalid
     Failure/Error: node = Builder::XmlMarkup.new(indent: 2)

NameError:
       uninitialized constant Builder::XmlMarkup
     # ./app/controllers/webui/patchinfo_controller.rb:123:in `save'
     # ./spec/controllers/webui/patchinfo_controller_spec.rb:194:in `block (4 levels) in <top (required)>'
     # ./spec/support/logging.rb:4:in `block (2 levels) in <top (required)>'
---
 src/api/app/controllers/webui/patchinfo_controller.rb | 2 ++
 src/api/script/start_test_backend                     | 1 +
 src/api/test/test_helper.rb                           | 1 +
 3 files changed, 4 insertions(+)

diff --git a/src/api/app/controllers/webui/patchinfo_controller.rb b/src/api/app/controllers/webui/patchinfo_controller.rb
index 6eb0e08c9..911408698 100644
--- a/src/api/app/controllers/webui/patchinfo_controller.rb
+++ b/src/api/app/controllers/webui/patchinfo_controller.rb
@@ -1,3 +1,5 @@
+require 'builder'
+
 class Webui::PatchinfoController < Webui::WebuiController
   include Webui::PackageHelper
   before_action :set_project
diff --git a/src/api/script/start_test_backend b/src/api/script/start_test_backend
index f99737349..f1b079998 100755
--- a/src/api/script/start_test_backend
+++ b/src/api/script/start_test_backend
@@ -3,6 +3,7 @@ ENV['RAILS_ENV'] ||= 'test'
 ENV['origin_RAILS_ENV'] ||= ENV['RAILS_ENV']
 ENV['LC_ALL'] = 'C'
 require File.expand_path(File.join(File.dirname(__FILE__), '..', 'config', 'environment'))
+require 'builder'
 
 include TestBackendTasks
 
diff --git a/src/api/test/test_helper.rb b/src/api/test/test_helper.rb
index a3b35280c..0affeb6c9 100644
--- a/src/api/test/test_helper.rb
+++ b/src/api/test/test_helper.rb
@@ -4,6 +4,7 @@ ENV['RAILS_ENV'] = 'test'
 
 require 'simplecov'
 require 'coveralls'
+require 'builder'
 require "minitest/reporters"
 
 Minitest::Reporters.use! Minitest::Reporters::SpecReporter.new
-- 
2.14.3

Places

File obs-server-2.8-0107-ci-Rails-5.1-Fix-test-failure-caused-by-missing-requ.patch of Package obs-server (Revision 617c434d5a327d4f9d9c86ae16dc58d1)

Places