jordi fita mas
97cf117da3
It made no sense to have a file upload in each form that needs a media, because to reuse an existing media users would need to upload the exact same file again; this is very unusual and unfriendly. A better option is to have a “centralized” media section, where people can upload files there, and then have a picker to select from there. Ideally, there would be an upload option in the picker, but i did not add it yet. I’ve split the content from the media because i want users to have the option to update a media, for instance when they need to upload a reduced or cropped version of the same photo, without an edit they would need to upload the file as a new media and then update all places where the old version was used. And i did not want to trouble people that uploads the same photo twice: without the separate relation, doing so would throw a constraint error. I do not believe there is any security problem to have all companies link their media to the same file, as they were already readable by everyone and could upload the data from a different company to their own; in other words, it is not worse than it was now.
83 lines
2.9 KiB
PL/PgSQL
83 lines
2.9 KiB
PL/PgSQL
-- Test add_home_carousel_slide
|
|
set client_min_messages to warning;
|
|
create extension if not exists pgtap;
|
|
reset client_min_messages;
|
|
|
|
begin;
|
|
|
|
set search_path to camper, public;
|
|
|
|
select plan(14);
|
|
|
|
select has_function('camper', 'add_home_carousel_slide', array['integer', 'text']);
|
|
select function_lang_is('camper', 'add_home_carousel_slide', array['integer', 'text'], 'sql');
|
|
select function_returns('camper', 'add_home_carousel_slide', array['integer', 'text'], 'integer');
|
|
select isnt_definer('camper', 'add_home_carousel_slide', array['integer', 'text']);
|
|
select volatility_is('camper', 'add_home_carousel_slide', array['integer', 'text'], 'volatile');
|
|
select function_privs_are('camper', 'add_home_carousel_slide', array['integer', 'text'], 'guest', array[]::text[]);
|
|
select function_privs_are('camper', 'add_home_carousel_slide', array['integer', 'text'], 'employee', array[]::text[]);
|
|
select function_privs_are('camper', 'add_home_carousel_slide', array['integer', 'text'], 'admin', array['EXECUTE']);
|
|
select function_privs_are('camper', 'add_home_carousel_slide', array['integer', 'text'], 'authenticator', array[]::text[]);
|
|
|
|
set client_min_messages to warning;
|
|
truncate home_carousel_i18n cascade;
|
|
truncate home_carousel cascade;
|
|
truncate media cascade;
|
|
truncate media_content cascade;
|
|
truncate company cascade;
|
|
reset client_min_messages;
|
|
|
|
insert into company (company_id, business_name, vatin, trade_name, phone, email, web, address, city, province, postal_code, country_code, currency_code, default_lang_tag)
|
|
values (1, 'Company 2', 'XX123', '', '555-555-555', 'a@a', '', '', '', '', '', 'ES', 'EUR', 'ca')
|
|
;
|
|
|
|
insert into media_content (media_type, bytes)
|
|
values ('text/plain', 'hello, world!')
|
|
, ('image/svg+xml', '<svg xmlns="http://www.w3.org/2000/svg" width="1" height="1"/>')
|
|
, ('image/x-xpixmap', 'static char *s[]={"1 1 1 1","a c #ffffff","a"};')
|
|
;
|
|
|
|
insert into media (media_id, company_id, original_filename, content_hash)
|
|
values (5, 1, 'text.txt', sha256('hello, world!'))
|
|
, (6, 1, 'image.svg', sha256('<svg xmlns="http://www.w3.org/2000/svg" width="1" height="1"/>'))
|
|
, (7, 1, 'cover4.xpm', sha256('static char *s[]={"1 1 1 1","a c #ffffff","a"};'))
|
|
;
|
|
|
|
insert into home_carousel (media_id, caption)
|
|
values (5, 'Previous caption')
|
|
;
|
|
|
|
select lives_ok(
|
|
$$ select add_home_carousel_slide(6, 'A caption') $$,
|
|
'Should be able to add a carousel slide with a caption'
|
|
);
|
|
|
|
select lives_ok(
|
|
$$ select add_home_carousel_slide(7, null) $$,
|
|
'Should be able to add a carousel slide without caption'
|
|
);
|
|
|
|
select lives_ok(
|
|
$$ select add_home_carousel_slide(5, 'New caption') $$,
|
|
'Should be able to overwrite a slide with a new caption'
|
|
);
|
|
|
|
select bag_eq(
|
|
$$ select media_id, caption from home_carousel $$,
|
|
$$ values (5, 'New caption')
|
|
, (6, 'A caption')
|
|
, (7, '')
|
|
$$,
|
|
'Should have all three slides'
|
|
);
|
|
|
|
select is_empty(
|
|
$$ select * from home_carousel_i18n $$,
|
|
'Should not have added any translation'
|
|
);
|
|
|
|
select *
|
|
from finish();
|
|
|
|
rollback;
|