Цветан обнови решението на 15.11.2016 21:04 (преди над 7 години)
+RSpec.describe 'Version' do
+ describe '.new' do
+ it 'creates an object with string' do
+ expect( Version.new('1.2.3.4')).to be_instance_of(Version)
+ expect( Version.new('0') ).to be_instance_of(Version)
+ expect( Version.new('') ).to be_instance_of(Version)
+ expect( Version.new ).to be_instance_of(Version)
Тези проверки кога могат да фейлнат? Възможно ли е Version.new да не е инстанция на Version?
+ end
+
+ it 'creates an object with instance' do
+ expect( Version.new(Version.new('1.2.3.4')) ).to be_instance_of(Version)
+ expect( Version.new(Version.new('0')) ).to be_instance_of(Version)
+ expect( Version.new(Version.new('')) ).to be_instance_of(Version)
+ expect( Version.new(Version.new) ).to be_instance_of(Version)
+ end
+
+ it 'doesn`t raise error' do
Това го тестваш в другите тестове. Този test case е излишен, защото не дава нищо ново.
+ expect { Version.new(Version.new('1.2.3')) }.to_not raise_error
+ expect { Version.new('1.2.3') }.to_not raise_error
+ expect { Version.new('1.2.0') }.to_not raise_error
+ expect { Version.new('') }.to_not raise_error
+ expect { Version.new('0') }.to_not raise_error
+ expect { Version.new }.to_not raise_error
+ end
+
+ it 'raises error' do
+ string = '1.'
Тази променлива защо ти е? Също, името string е почти толкова лошо, колкото и tmp и obj :)
+ message = "Invalid version string \'#{string}\'"
+ expect { Version.new(string) }.to raise_error(ArgumentError, message)
+ string = '.3'
+ message = "Invalid version string \'#{string}\'"
+ expect { Version.new(string) }.to raise_error(ArgumentError, message)
+ string = '0..3'
+ message = "Invalid version string \'#{string}\'"
+ expect { Version.new(string) }.to raise_error(ArgumentError, message)
+ string = '3,4'
+ message = "Invalid version string \'#{string}\'"
+ expect { Version.new(string) }.to raise_error(ArgumentError, message)
+ string = 'abc'
+ message = "Invalid version string \'#{string}\'"
+ expect { Version.new(string) }.to raise_error(ArgumentError, message)
+ string = 'a.b.c'
+ message = "Invalid version string \'#{string}\'"
+ expect { Version.new(string) }.to raise_error(ArgumentError, message)
+ end
+ end
+
+ describe '#<=>' do
+ it 'compares versions' do
+ expect(Version.new('1.1.0')).to eq(Version.new('1.1'))
+ expect(Version.new('1.1')).to eq(Version.new('1.1'))
+ expect(Version.new('0.1')).not_to eq(Version.new('1'))
+ expect(Version.new('1.2.3')).to be < Version.new('1.3')
+ expect(Version.new('1.2.3')).to be < Version.new('1.3.1')
+ expect(Version.new('')).to eq(Version.new)
+ end
+ end
+
+ describe '#to_s' do
+ it 'converts to string' do
+ expect(Version.new('1.1.0').to_s).to eq('1.1')
+ expect(Version.new('1.1').to_s).to eq('1.1')
+ expect(Version.new('').to_s).to eq('')
+ expect(Version.new.to_s).to eq('')
+ expect(Version.new(Version.new('1.2.3')).to_s).to eq('1.2.3')
+ end
+ end
+
+ describe '#components' do
+ it 'breaks into components' do
+ expect(Version.new('1.3.5').components).to match_array([1, 3, 5])
Каква е разликата между eq и match_array?
+ expect(Version.new('1.3.5.0').components).to match_array([1, 3, 5])
+ expect(Version.new('1.0.0.0').components).to match_array([1])
+ expect(Version.new('').components).to match_array([])
+ end
+
+ it 'breaks into N components' do
+ expect(Version.new('1.3.5').components(1)).to match_array([1])
+ expect(Version.new('1.3.5').components(2)).to match_array([1, 3])
+ expect(Version.new('1.3.5').components(3)).to match_array([1, 3, 5])
+ expect(Version.new('1.3.5.0').components(3)).to match_array([1, 3, 5])
+ expect(Version.new('1.3.5').components(4)).to match_array([1, 3, 5, 0])
+ expect(Version.new('1.3.5').components(0)).to match_array([1, 3, 5])
+ expect(Version.new('').components(4)).to match_array([0, 0, 0, 0])
+ end
+ end
+
+ describe 'Version::Range' do
+ describe '.new' do
+ it 'valid input' do
+ expect(Version::Range.new(Version.new('1.0.0'), Version.new('2.0.0')))
+ .to be_instance_of(Version::Range)
------- || -------
+ expect(Version::Range.new('1.0.0', '2.0.0'))
+ .to be_instance_of(Version::Range)
+ end
+
+ it 'invalid input' do
+ string = '1.0.0.'
------- || -------
+ message = "Invalid version string \'#{string}\'"
+ expect { Version::Range.new(Version.new(string), Version.new('2.0.0')) }
+ .to raise_error(ArgumentError, message)
+ expect { Version::Range.new(string, '2.0.0') }
+ .to raise_error(ArgumentError, message)
+ end
+ end
+
+ describe '#include?' do
+ it 'includes an instance' do
+ range = Version::Range.new(Version.new('1'), Version.new('2'))
+ expect(range.include?(Version.new('1.5'))).to be true
+ end
+
+ it 'includes a string' do
+ range = Version::Range.new(Version.new('1'), Version.new('2'))
+ expect(range.include?('1.5')).to be true
+ end
+ end
+
+ describe '#to_a' do
+ it 'includes first and exludes last' do
Това също е излишно, защото го тестваш долу в it lists all versions
+ range = Version::Range.new(Version.new('1.1.0'), Version.new('1.2.2'))
+ expect(range.to_a).to include('1.1')
+ expect(range.to_a).not_to include('1.2.2')
+ end
+
+ it 'doesn`t list versions for equal ones' do
+ range = Version::Range.new('1.1', '1.1')
+ expect(range.to_a).to eq([])
+ end
+
+ it 'lists all versions' do
+ expect(Version::Range.new('1.1.0', '1.2.2').to_a).to eq \
Това е много гадно подреждане. Направи го така:
expect(Version::Range.new('1.1.0', '1.2.2').to_a).to eq [
'1.1', '1.1.1', '1.1.2', '1.1.3', '1.1.4', '1.1.5',
'1.1.6', '1.1.7', '1.1.8', '1.1.9', '1.2', '1.2.1'
]
+ [
+ '1.1', '1.1.1', '1.1.2', '1.1.3', '1.1.4', '1.1.5', \
Защо реши, че ти трябва тази наклонена черта?
+ '1.1.6', '1.1.7', '1.1.8', '1.1.9', '1.2', '1.2.1'
+ ]
+ expect(Version::Range.new('1.1', '1.2').to_a).to eq \
+ [
+ '1.1', '1.1.1', '1.1.2', '1.1.3', '1.1.4', '1.1.5', \
+ '1.1.6', '1.1.7', '1.1.8', '1.1.9'
+ ]
+ end
+ end
+ end
+end
За този метод необходимо ли е да тествам всички негови производни оператори? ( ==, >, <, >=, <=)
Да. Иначе няма как да си сигурен, че работят.
Адекватен начин ли е това да се провери дали не сме повредили инстанцията, след като сме й викнали #components?
Идеята не е да провериш
дали components поврежда инстанцията като се викне, адали components може да се използва, за да се промени инстанцията.Освен това,
object_id-то на инстанцията никога няма да се промени. Каквото и да й правиш.Промени решението, така че да няма
dup, зареди файла в едно irb и се опитай да пресъздадеш проблема. После го направи на тест.